m_sajoin Abort and report if the target user is already on the channel

Fixes issue #820 reported by @UselessOper

m_spanningtree Fix routing of unicast messages

Kill <channels> in favour of <connect:maxchans> and <oper:maxchans>.

Remove channels/high-join-limit privilege

Implement support for numeric character references.

Clean up a bunch of legacy code in the Poll socketengine.

Remove various broken/useless features from the build system.

- Removed support for changing the build directory using configure.
  This can still be set using make BUILDPATH=foo.
- Removed support for SPLIT_CC builds. This is not documented
  anywhere and is quite useless as it doesn't work.
- Remove STARTSCRIPT from configure; always install the perl
  helper.

m_filter Allow exemption of messages sent to nicks specified in <exemptfromfilter> tags

Issue #655

Redirect the reader to filters.conf.example for advanced filter configuration

m_filter Rename config key used in exemptfromfilter from "channel" to "target", but keep compatibility

m_filter Rename exemptfromfilter field to exemptedchans, use std::set::count()

m_filter Use case insensitive compare for exempted targets, add typedef for the set

m_services_account Remove now needless workaround added to 2.0 to display the correct host when cgiirc users log in using SASL

Fix m_cgiirc to set the user's host straight away when using WEBIRC.

Avoid double Membership lookup in Channel::UserList()

The user is always inside if UserList() is called from ForceJoin() and the HasUser() result obtained in the /NAMES handler can be reused

Move checks determining whether a user is allowed to view the NAMES list of a channel from Channel::UserList() to cmd_names

Eliminate usage of has_one in Channel::UserList()

m_blockcaps Get rid of needless cast, use unsigned types

m_blockcaps Simplify CTCP ACTION (/me) detection

m_spanningtree Don't rely on "m_sha256.so" being loaded, use any "hash/sha256" service provider

m_spanningtree Remove the (now) undocumented disablehmac config option

Fix accessibility in intrusive_list_node and return value of operator-- in intrusive_list

Merge insp20

Release 2.0.16

Fix IPv6 address handling in 6aaf7047297b739377e7d509cb914f32447fb281

Fix typo in helpop-full that prevented it from loading and other minor helpop issues

Fix snotice about disabled command usage being sent to the wrong snomask

Add REG_ALL checks to treat unregistered users as nonexistent in more cases

Fix typos and update text in the configure script

Some of these were picked from #448

Major example configuration and helpop update

Big thanks to @m4z (#621) and @Robby- (#630) who contributed a lot to this

m_override Deduplicate code

Remove unused User::WriteCommonExcept()

WriteCommonRaw(msg, false) has similiar functionality with a std::string parameter

Move User::SendAll() into core_privmsg

This functionality is only used by the PRIVMSG and NOTICE handlers

Remove User::WriteTo() functions; use User::WriteFrom() instead

The removed functions were rarely used and all they did was to call User::WriteFrom() with swapped arguments. On call sites, WriteTo() hardly offers any readability improvement over WriteFrom().

Check if Membership::GetPrefixChar() returns 0 before appending it to a string

Spotted by @barosl

Build coremods on Windows, add general support to the cmake script for building modules consisting of multiple compilation units

Instead of special casing m_spanningtree create one project for each subdir in src/modules/ and src/coremods/

Convert the global list of dynamic reference pointers to an intrusively linked list

m_watch Validate targ before use, noticed while rewriting m_watch for 2.2

m_globalload Don't pass a callback to ModuleManager::Reload() if reloading m_globalload

m_httpd Fix typo noticed by @SaberUK

Change storage of UserManager::all_opers to be a vector

Add typedef OperList to UserManager for use with all_opers

Add stdalgo::delete_all() that deletes all elements in a container

Cull ident sockets instead of immediate delete, add stdalgo::culldeleter

Make it possible to customize how SimpleExtItem should delete items

Create the stdalgo namespace for container-related algorithms, add stdalgo::vector::swaperase()

m_operprefix Don't set +y on the oper if hideoper is being unset due to deopering

Fixes issue #801 reported by @UselessOper

Rewrite clone counting to use one map instead of two

m_connectban Don't depend on the definition of clonemap

Fix invalid iterator use when removing modes in ModuleManager::DoSafeUnload()

Change allocation of InspIRCd::Timers to be physically part of the object containing it

Change allocation of UserManager::clientlist to be physically part of the object containing it

Add UserManager::GetUsers()

Change allocation of UserManager::uuidlist to be physically part of the object containing it

Change allocation of InspIRCd::chanlist to be physically part of the object containing it

Add InspIRCd::GetChans(), remove ChannelCount()

m_permchannels Remove pointless cleanup ceremony in cull()

Fix NAMES list getting truncated on servers having an excessively long server name

Backport "Refactor Channel::UserList() to use std::string"

This gets rid of questionable snprintf() usage, reported by @hifi

Backported commit: 8a06d54076551387f83a29360478ee6605e241b6

Deduplicate and move the *MatchesEveryone() functions to core_xline

cmd_names Allow opers having the channels/auspex priv to view the NAMES list of +s channels from outside

A version of the check removed by this commit is in Channel::UserList() which takes the priv into account

Discovered by @Cronus89

Improve detection for non-cidr masks to prevent unwanted matches

Fixes issue #762 reported by @neoinr

cmd_quit Display quit messages of remote users

Move the wallops mode from the core into core_wallops

Make the maximum hostname length configurable in the config.

Fix m_permchannels not prepending the path when reading the config.

Move admin settings into core_info

Move {prefix|suffix|fixed}{quit|part} into core_user

Read the die and restart password and their hash type on demand

Update detection in module loading commands that relied on old command module names

Move UNLOADMODULE handler into core_loadmodule

Create the core_stub module

This handles CONNECT, LINKS, SERVER and SQUIT

Load core_*.so instead of cmd_*.so

Integrate coremods into the build system

Create the core_xline module

Create the core_user module

Create the core_oper module

Create the core_info module

Create the core_channel module

Move src/commands/cmd_*.cpp to src/coremods[/core_*]/

Remove out of date doc and fix typo in commands/cmd_*.cpp

m_spanningtree Remove remains of the KeepNickTS workaround

Update nick timestamps in User::ChangeNick(), pass the new TS in a parameter

m_spanningtree Do pointer comparison before deleting one of the pointers in TreeSocket::Squit()

While the previous code worked fine in practice, it was incorrect in theory

m_spanningtree Don't send snotices to servers about remote servers splitting

The snotice is sent to opers when the SQUIT is processed by their server

Fix NULL dereference on rehash when there is a fatal error in the configuration of connect classes

Fixes issue #780 reported by @JDowny

Merge pull request #784 from ShutterQuick/patch-1

Fix build problem on Solaris

Merge pull request #783 from SaberUK/insp20+solaris-hates-posix

Fix Solaris not being able to find modules without pkg-config.

Fix build problem on Solaris

-rpath only takes one dash
Fixes the Solaris build \o/

'''
<fraggeln> SaberUK: http://m.theo.nu/jenkins/job/InspIRCd-2.0/435/label=solaris/console <--
<fraggeln> do you have any clue on why?
<@Shutter> Hmm. Where did that extra - come from.
<fraggeln> Shutter: no idea :)
[...]
<fraggeln> Shutter: sub make_rpath($;$) <-- in make/utilities.pm
<fraggeln> thats where that extra - is'''

Fix Solaris not being able to find modules without pkg-config.

Remove REG_ALL checks from User::SharesChannelWith()

These checks are never true currently and even if one or both users are unregistered proceeding is harmless

cmd_topic Remove redundant NULL check

Give m_connectban a configurable ban message

Simplify /SAJOIN syntax and add permission for joining other users.

m_chanfilter Remove needless mode parameter is empty check

m_cycle Remove redundant IS_LOCAL() check

cmd_kill Remove now needless check that prevented KILLing the same user twice

cmd_kill Hide server notices if the source is ulined

Issue #742

m_svshold Hide server notices by default

Issue #742

Throw an exception if ModeParser::AddMode() fails

Moving CGIIRC to a new snomask (w)