Remove empty Thread destructor

threadengine_win32 Don't ask for the thread id of newly created threads pointlessly

Move and rename class ThreadData to ThreadEngine::ThreadState

Move and rename ThreadData::FreeThread() to ThreadEngine::Stop() and document what it does

Make Thread::state private

Change allocation of ThreadData to be physically part of the object containing it

Remove empty ThreadEngine constructors and destructors

Remove the unused ThreadEngine::GetName() method from both implementations

Change allocation of InspIRCd::Users to be physically part of the object containing it using fakederef

Change allocation of InspIRCd::SNO to be physically part of the object containing it using fakederef

Remove now needless dynref_init_complete

ModuleManager is constructed when ServerInstance is non-NULL, test that instead

Change allocation of InspIRCd::Modules to be physically part of the object containing it using fakederef

Change allocation of InspIRCd::Logs to be physically part of the object containing it using fakederef

Change allocation of InspIRCd::Modes to be physically part of the object containing it using fakederef

Add the fakederef class

Change all occurrences of plain sort() to std::sort()

core_userhost Show real host if the target is the same as the user doing the /USERHOST

core_userhost Append data to the output in a saner way

core_userhost Do the HasPrivPermission() check only once, not once per nick

core_ison Use iterators

core_ison Extract duplicated code into a function

Change append(" ") to push_back(' ')

core_ison Truncate string after sending it on overflow instead of recreating it

core_ison Don't deduplicate nicks

Deduplication makes us do pointless processing for every ISON and offers
no benefit at all to proper clients that send a nick only once

core_ison Remove check that is always true

core_list Improve readability by assigning the Channel being inspected to a variable

core_list Check whether the chan name/topic has to be Match()ed once, not once per chan

core_list Do the HasPrivPermission() check only once, not once per chan

Check Q-Lines on nick change in core_xline

Reject nickname case changes too when enforcing <security:restrictbannedusers>

Change the type of the user parameter in the OnUserPreNick() hook from User to LocalUser

No remote users were passed to this hook before.

Remove needless IS_LOCAL() checks.

Increment serverstats::Collisions when a collision is handled, not when a module denies a nick change

m_spanningtree Share server description updates via SINFO desc

Update the description field in the Server object representing the local server on rehash

m_spanningtree Remove server-to-server VERSION handler

m_spanningtree Translate SINFO version to VERSION and vice versa for 1202 protocol servers

m_spanningtree Exchange public server version strings via SINFO instead of VERSION

m_spanningtree Show the full version string of servers to opers

Issue #637, suggested by @CuleX

m_spanningtree Send and receive full version strings via SINFO

m_spanningtree Add a field to TreeServer for storing full version strings

m_spanningtree Add server-to-server SINFO command handler and builder

Don't send SINFO to 1202 protocol servers

m_spanningtree Add TreeSocket::SendServerInfo() that sends all additional data about a server

Kill needless #includes in source files

Remove unused #define ERROR -1

m_spanningtree Remove declaration for nonexistent function: OnLine()

m_spanningtree Use vector back() where it makes sense

m_spanningtree Initialize boolean member variables in the constructor of the module

m_spanningtree Initialize TreeServer::UserCount for TreeRoot in its constructor

Make the default ProtocolInterface instance part of class InspIRCd

The protocol module no longer has to delete it on load and construct it on
unload, only assign a new value to ServerInstance->PI

m_spanningtree Change allocation of the specialized ProtocolInterface to be physically part of ModuleSpanningTree

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

Remove global Commandtable typedef

Make CommandParser::cmdlist private

Add CommandParser::GetCommands() and typedef CommandMap and use it instead of directly accessing cmdlist

Use CommandParser::GetHandler() in ServerConfig::ApplyDisabledCommands()

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

Pull in bancache.h from inspircd.h

Move the definition of the BanCacheHit constructor into the source file from the header

Move BanCacheHash typedef into the private part of BanCacheManager

Change allocation of BanCacheManager::BanHash to be physically part of the object containing it

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

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

While at it, remove "stats" from all field names

m_spanningtree Explicitly specify the routing for RESYNC as the one inherited from ServerCommand doesn't suit it

Command::GetRouting() defaulted to local only when RESYNC was added, but
ServerCommand overrides it.

m_spanningtree Strip membership id from KICKs sent to 1202 protocol servers

Send the membership id when kicking a remote user and drop KICKs with mismatching membership ids

This fixes the desync happening when a PART+JOIN crosses a KICK targetting
the same user.

m_spanningtree Strip membership ids from FJOINs sent to 1202 protocol servers

m_spanningtree Send and parse FJOINs with membership ids

m_spanningtree Add CmdBuilder::push_raw_int()

m_spanningtree Use the FJOIN builder when a channel is created

m_spanningtree Add CmdBuilder specialization for FJOIN

m_spanningtree Send, parse and translate IJOINs with membership ids

m_spanningtree Assign an id to new Memberships

m_spanningtree Initialize membership ids on load

Add foundation for membership ids

Add ConvToUInt64()

Return the newly created Membership object from Channel::ForceJoin()

Add channel TS to server-to-server INVITE to detect and drop unauthorized invites

The syntax of the server-to-server INVITE command changes from
:<source> INVITE <target> <channel> [<expire>]
to
:<source> INVITE <target> <channel> <chants> [<expire>]

m_spanningtree Allow callers of InsertCurrentChannelTS() to customize positions

cmd_kick Assign a value to the kick reason string more reasonably

Use the iterator version of Channel::KickUser() in a few places

Change Channel::KickUser() to accept an iterator, add overload that accepts a User

Remove srcmemb parameter

Move pre-kick checks from core to cmd_kick (core_channel)

Document the config options of m_remove

m_remove Make the protected rank configurable instead of hardcoding it to 50000

Change type of ulevel and tlevel to unsigned int

m_remove Recognize /REMOVE <chan> <nick>

The old syntax is now deprecated

Fixes issue #860 reported by @Shawn-Smith

m_remove Manually forward all REMOVE messages, turn FPART into REMOVE

Use std::find() in ParseStack::ParseFile()

Deduplicate ParseStack::ParseFile() and ParseExec()

m_ssl_openssl Remove unused MAX_DESCRIPTORS define

Change the number reported by SocketEngine::GetMaxFds() to be informal

Do not exit if we can't determine it

socketengine_epoll Pass a dummy value to epoll_create() that does not depend on GetMaxFds()

From man epoll:
Since Linux 2.6.8, the size argument is unused, but must be greater than zero. (The kernel dynamically sizes the required data structures without needing this initial hint.)

Relax fd bounds checking

We can cope with fds greater than SocketEngine::GetMaxFds() since 3752b3f59d5216d7dc6221a361efc76b9ad2273d

m_channames Remove temporary vector from ValidateChans()

m_channames Call ValidateChans() from cull() on unload

This ensures that if ValidateChans() removes everyone from a channel the
channel is added to the cull list at the right time, that is, before the
destruction of objects begin.

The core IsChannel() is less strict than us regardless of our
configuration (for now) but nevertheless this is the correct behavior.

m_ident Close the ident socket when the object is culled instead of using hooks

m_ident Make use of the min parameter of ConfigTag::getInt()

Describe <ident:nolookupprefix>

m_ident: Add an option to allow idents of users to still be prefixed with a '~' for connect classes which have disabled ident lookups through the <connect:useident> setting.

Fixes #683.

Some changes by @attilamolnar, original PR #684

m_ssl_gnutls Don't include cap.h

The tls cap is handled by m_starttls

Switch to std::string::compare() from substr() in a couple of places

Prefix all definitions in config.h to avoid potential collisions.