more perl inheritance warning fixes

Merge pull request #365 from ailin-nemui/perl_nick

fix nick class hierarchy

Merge pull request #364 from ailin-nemui/no_cap_sasl

stop cap_sasl

fix nick class hierarchy

stop cap_sasl

Merge pull request #360 from LemonBoy/backport-patch

Save a patch from the Flyspray oblivion, plus a nice unrelated commit to make everyone (un)happy

Merge pull request #359 from LemonBoy/dcc-assert

Let ignore_check do its work when server is NULL

Merge pull request #361 from LemonBoy/awaylog-format

Use the expanded filename when picking the awaylog

Use the expanded filename when picking the awaylog

This fixes a long-standing bug where 'fname' was being feed to cat
instead of 'real_fname', causing it to quit with a 'No such file or directory'
error.

FS#377

Use 'isblank()' instead of a custom macro

Correct a wrong use of the 'paste_buffer' variable

The function "static void paste_buffer_join_lines(GArray *buf)" in
"src/fe-text/gui-readline.c" is supposed to join lines from the GArray
pointed to by *buf under certain circumstances.

In the code of the function "buf" is actually used for getting the length
of the GArray, but to get a pointer to the data, "paste_buffer->data" is
used; paste_buffer is defined in the scope of the whole file.

This delivers the desired result, because this function is only called
once, with "paste_buffer" as the argument. If paste_buffer_join_lines()
will ever be used with a different argument, it will fail.

Let ignore_check do its work when server is NULL

A NULL-check has been added to the ignore_match_server macro, making the
function safe from a pointer perspective.
Fixes #193 in the meanwhile.

Correctly alias 'channel' to '#channel'

Use the same approach used in 'irc_channels_join'.
Remove 'irc_nick_strip' since it was unused.

Merge pull request #350 from LemonBoy/fix-99

Strip trailing whitespace from commands

Merge pull request #351 from vague666/hilight_default_behaviour

Fix /hilight list output behaviour depending on enabled flags

/hilight list print output with enabled flags, except when -word and -nick are used

Use the PARAM_FLAG_STRIP_TRAILING_WS flag wherever possible.

Strip the trailing whitespace from /join commands.

Fixes #99 for great good.

Add an option to strip trailing whitespace when parsing commands

Merge pull request #313 from isundil/master

Fix #45 Make it easy to delete default channels, servers and networks

Merge pull request #341 from dequis/strsplit-len-fix

Fix invalid reads in strsplit_len when splitting on spaces

strsplit_len: make it look more like the original version

strsplit_len: use strlen() directly instead of a remaining_len variable

Merge remote-tracking branch 'origin/master' into bracketed-paste

Merge pull request #330 from dequis/xterm-keypad-enter

Add xterm's keypad enter, meta-O-M to "key return" bindings

Merge pull request #320 from irssi/ahf/add-setting-type-any

Fix warnings

Merge pull request #344 from rrebello/m4-quote-fix

Fix quote around macro argument

Merge pull request #345 from LemonBoy/sasl-reconnect

Preserve the sasl_ options across reconnects.

Preserve the sasl_ options across reconnects.

Fix quote around macro argument

In m4/curses.m4, line 134, the 5th argument passed to AC_NCURSES was
surrounded by '"' instead of '[' and ']'. Because of that, the expansion
of AC_NCURSES in that case would produce the following line inside the
configure script (note the repeated double quotes):

  screen_manager=""ncurses on $withval/include""

That would cause the following error when configure was executed with
the "--with-ncurses=dir" argument:

  ./configure: line 13468: on: command not found

Although in the case above the error doesn't actually influence the
build process ('screen_manager' isn't used anywhere in the script),
trying to execute 'on' might be harmful if it corresponded to an
existing command in the user's environment.

Fix invalid reads in strsplit_len when splitting on spaces

The symptom for this one is randomly getting lines split before the last
word, even if there's no need for splitting. Also, this function is only
reached if recode is on, and iconv failed (for example, due to an
incorrect source charset). Thanks to vague for finding this and
providing valgrind logs.

The loop that looks for spaces tried to read backwards from the end of
the current line, with the end being determined by len. Assuming
strsplit_len() with len=400, this meant accessing str[399] in the first
iteration. For strings that don't need splitting, this means an invalid
read always.

If that invalid read happens to hit garbage that has a space character,
(len - offset) points after the end of string, which isn't a problem for
g_strndup() since it stops at the first null, and no splitting happens.
If the garbage doesn't have any spaces, it splits by the last word.

This commit avoids that loop entirely if (remaining_len > len). It also
changes the way it iterates over the string to be much less confusing.

Merge pull request #333 from dequis/recon-tag-fix

Fix /reconnect RECON-1 saying "Reconnection tag 1 not found"

Fix /reconnect RECON-1 saying "Reconnection tag 1 not found"

Turns out it was fixing the wrong string, and trying to do
atoi("RECON-1") instead of atoi("1").

"/reconnect 1" worked, but "/reconnect RECON-1" gave that confusing
error message.

Add xterm's keypad enter, meta-O-M to "key return" bindings

From the 'kent' terminfo entry. Also applies to putty.

Fixes #327

Merge pull request #318 from LemonBoy/fix-utf8-elements

Fix the display of utf8 sequences in the gui

Fix the indentation.

Merge pull request #314 from LemonBoy/sasl_session

Save the sasl state in the session

Add -Wall and -Werror as CFLAGS to make.

Remove check for >= 0 for unsigned unichar.

Add SETTING_TYPE_ANY and replace -1 with it.

Simplify Travis config.

Merge pull request #322 from LemonBoy/fix-warning-awaylog

Include write-buffer.h in log-away.c

Include write-buffer.h in log-away.c

Silence a warning and make the world a better place.

Set HOST_NAME_MAX to 255, if it's undefined.

Thanks to Jilles and dx.

Fixes #309

Use silent rules.

Add missing null terminator to the g_build_path() varargs

Lemon broke it a few commits ago.

Merge pull request #316 from LemonBoy/fix_mem_leak

Fix a memory leak.

Merge pull request #319 from LemonBoy/awaylog-flush

Flush the dirty buffer to disk

Flush the dirty buffer to disk

Given a big enough write_buffer_size and a long enough
write_buffer_timeout it might be possible to show the user an incomplete
or empty awaylog.

Patch by: Petteri Aimonen

Kill an unneeded declaration

Even simpler logic

Rework the logic to avoid allocating memory

Fix the display of utf8 sequences in the gui

term_addstr() had a long-standing fixme that suggested it didn't
take into account the string encoding when calculating the string
length.
The BIG5 code path is untested.

Fix a memory leak.

g_get_current_dir() returns a heap-allocated string.

Merge pull request #294 from dequis/key-states-rescan-recursion-limit

Limit recursion depth of key/combo expansion in key_states_scan()

Merge pull request #315 from LemonBoy/prevent_blank_usermode

Don't set the usermode field if blank

Don't set the usermode field if blank

Fixes FS#919

Save the sasl state in the session

This is seemingly required to have irssi re-authenticate after a
restart.

Fix return value of server_setup_remove_chatnet

Updated server removal
Removing network will also remove attached channels

Fix #45 Make it easy to delete default channels, servers and networks
Removing network will now also remove all attached servers

Merge pull request #312 from Jellyfrog/patch-1

Switch to modern Travis CI infrastructure

Switch to modern Travis CI infrastructure

Fixes #310

Merge pull request #308 from vague666/sasl_in_network_list

Add sasl info to /network list output if available

Make sure sasl settings are defined before printing them out

Add sasl info to /network list output if available

Merge pull request #307 from chi6rag/master

Remove Typo from Startup FAQs

Remove Typo from FAQs

Handle empty bracketed pastes (or sequences of those)

Both cases were off-by-one mistakes erring on the side of being too
conservative. This fixes these two harmless issues:

- For a single empty paste, it required another keystroke before
  processing it
- For a sequence of themcase, a single '~' was left in the input

Handle a paste start marker right after an end one (ignore both)

This actually workarounds a bug with the "st" terminal, for which i've
already submitted a patch, but irssi needs to be able to handle it
decently too.

Create paste_bracketed_middle() function to handle small pastes

"Small" as in ending in the same sig_input() call where they started

Fix some minor style issues

Send last line of bracketed paste together with the rest

paste_bracketed_end: Fix rest length calculation

Save the part of the paste buffer after the bp_end marker for later

Also move relevant code to a paste_bracketed_end() function

Improve bracketed paste start/end detection

- Use a keybinding to detect the start of a bracketed paste
- Iterate over the paste buffer looking for the end marker

Get rid of the non-portable memmem

The sequences we're after are found at the beginning or at the end of the
buffer, there's no need to scan the whole thing.

Enable the bracketed paste mode on demand

Replace some hairy logic with g_array_remove_range

In the hope it'll do the same under the hood.

Toggles

Implement the bracketed paste mode

As an alternative method of paste detection, more reliable but might not be
supported by all the VTs.

Merge pull request #305 from dequis/install-doc-mention-local-lib

INSTALL: mention local::lib for home directory installations

INSTALL: mention local::lib for home directory installations

I had problems installing this and this file didn't have the answer.
So I'm adding what nei told me to do.

Merge pull request #304 from LemonBoy/sasl_user_options

Allow the user to set and modify the SASL parameters

Allow the user to set and modify the SASL parameters

The /NETWORK ADD command now is able to modify the SASL mechanism, the
username and the password on a chatnet basis.

Merge pull request #302 from vague666/rawlog_save_with_dir

Added functionality to create dir if dir specified in rawlog filename

Applied patch from fs#275 to make /hilight -mask -line work properly

Added functionality to create dir if dir specified in rawlog filename

Merge pull request #301 from dequis/dcc-get-ntfs-temp-bug

dcc-get: close() the temp fd so we don't get ETXTBSY in ntfs mounts

dcc-get: close() the temp fd so we don't get ETXTBSY in ntfs mounts

Patch from debian bug 696963 [1]

Fixes github bug #220 and flyspray bug 867 [2]

[1]: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=696963
[2]: http://bugs.irssi.org/index.php?do=details&task_id=867

Merge pull request #298 from dequis/multi-prefix

Add multi-prefix to list of capabilities to request

Merge pull request #300 from dequis/drop-centos5

Drop centos 5 support, bump glib requirement to 2.16

Drop some glib version checks that are not needed anymore

The g_strcmp0 fallback in particular was broken since it was used in a
few places as a GCompareFunc, and macros don't work that way.

Yes, that one was my fault, but nobody complained :D

Bump glib2 requirement from 2.6 to 2.16

This drops support for rhel/centos 5 (18 months left of its 10 year
support cycle)

Keeps support for debian 5.0 (lenny) and ubuntu 8.04 LTS, both of
which are unsupported, so bumping up to glib 2.24 wouldn't be a problem,
but it's not needed atm.

Revert "Revert "Bug fix - docdir is ignored during installation.""

This reverts commit d222c11f74a5662df59e00e7ae6c8105179a804e.

Merge pull request #264 from ailin-nemui/window-help

Improve the WINDOW help

Improve the WINDOW help

Properly describe the window command.
Make help sort order deterministic.

Add multi-prefix to list of capabilities to request

Turns out event_names_list() in irc-nicklist.c already handles this.

event_who() just ignores it, which is probably a good idea since some of
the irc servers I tested this with have a bug that results in sending
multiple prefixes in the NAMES reply but not in the WHO one (they were
forks of ircd-hybrid before 7.3.0)

And NAMES always happens, anyway. WHO is omitted sometimes for huge
channels.

sig_message_irc_op_public: fix nickmode lookup, use cleantarget instead

Merge pull request #297 from vague666/hilight_nick_matches

Add new setting to modify behaviour of hilight_nick_matches to match anywhere in message

Fixes #56