Merge pull request #608 from SaberUK/master+buildprop-dedupe

Clean up the build system properties and related code.

Clean up the build system properties and related code.

- Deduplicate getcompilerflags, getdependancies, getlinkerflags.
- Remove $NoPedantic (add -Wno-pedantic to $CompileFlags instead).
- Remove --enable-freebsd-ports-openssl and all related code (this
  will be replaced with --no-pkg-config=[name] in the future).
- Remove some unused build system properties.
- Remove support for caching third party include and library paths
  (can cause unexpected problems when they change).

Remove $ModDep comments

Store oper types and opers in separate containers

m_ssl_gnutls Fix feature testing and cipher suite related errors on older GnuTLS versions

Use the (old) LIBGNUTLS_VERSION_* defines as a fallback when GNUTLS_VERSION_* is unavailable

Fixes #595 reported by @rhylan

Clean up a few constructors

Do not silently correct a zero TS in Channel::Channel(); require callers to supply a valid TS instead

cmd_whowas Major cleanup, fix inefficiencies

Add ConfigTag::getDuration() with optional bounds checking

Merge pull request #607 from SaberUK/master+match

Clean up wildcard code.

Clean up wildcard code.

- Move references and pointers next to the type.
- Rename match_internal to MatchInternal.
- Remove duplicate NULL check on map in MatchCIDR.

Merge pull request #591 from SaberUK/master+config-tweaks

Various patches relating to configuration.

Add <path> keys for all of the compile-time paths.

Also, make all paths in the config relative to their associated
directory. This reverts a change in 2.0 which turned out to be a
terrible idea, especially for system-wide installations.

Require privs given on initial join to be in <options:defaultmodes>.

Rename <options:moronbanner> to <options:xlinemessage>.

This name is more descriptive of what is actually is used for.

Replace range() with min and max arguments on getInt().

Prevent servers from ulining themselves.

Almost every single case I have seen where this has been done has
been an accident.

Change the default for <security:customversion> to an empty string.

RFC 1459 says this field is for comments about the version. If the
operator has no comments then we should not show any.

Warn if the server has no <bind> blocks.

Use an example server name if <server:name> is not defined.

Rename <connect:nouserdns> to <connect:resolvehostnames>.

This name is more descriptive and avoids a double negative which
could make the name of this setting unclear to non-native speakers.

Remove virtual User::GetClass() (but keep LocalUser::GetClass())

Accept std::string as parameter in User::ChangeHost(), ChangeIdent() and ChangeName()

Pass the string itself to IsIdent() instead of string.c_str()

Move host cycle logic into m_hostcycle

Fix a couple of issues

- Wrong doc for Snomask::GetDescription()
- Incorrect idle time when WHOISing remote users
- String + int = garbage
- Remote MODE commands (user mode changes) being broadcast in addition to being forwarded
- Incorrect revision being shown

Issue #604, fix m_dnsbl, broken in accccc212cd4f08a3c5532b1ae7a17e76bac8718

Call ModeParser::Process() directly instead of going through the CommandParser

Simplify the way how m_conn_umodes assembles the modes it sets

FakeUsers are only inserted into UserManager::uuidlist; don't try to erase() them from clientlist

Merge pull request #599 from SaberUK/master+configure-tests

Refactor duplicate test code into a run_test subroutine.

Refactor duplicate test code into a run_test subroutine.

Automatically attach modules to events

Change the syntax of FOREACH macros to be less dumb.

Merge pull request #590 from SaberUK/master+module-logging

Change modules to use the MODNAME constant.

Modify the log message to contain the log type.

Change httpd modules to use the MODNAME constant in headers.

Change modules to use the MODNAME constant when logging.

The majority of modules were logging with their module name as the
log type. There was a few places which were logging to a non-name
type but, with the exception of CONFIG, those messages are so
uncommon that it doesn't make sense to use a seperate type for
them.

Remove a few not-so-useful debug messages

Merge pull request #596 from SaberUK/master+genssl-auto

Add 'auto' option to genssl to automatically select a generator.

Merge pull request #593 from SaberUK/master+configure-shuffle

Remove a ton of duplicate and obsolete code from configure.

Add 'auto' option to genssl to automatically select a generator.

Remove a ton of duplicate and obsolete code from configure.

- Add the data and log paths to dumphash.
- Always exit with EXIT_FAILURE on error.
- Always replace the old config.h with our new one.
- Clean up the ./configure --help message slightly.
- Install dhparams.pem into the conf directory on GnuTLS as well as
  OpenSSL.
- Only show the users InspIRCd Git revision when building from Git.
- Remove a check for MinGW (we have proper Windows builds).
- Remove a check for FreeBSD 4 (released in 2000).
- Remove a ton of pointless tab-indented comments.
- Remove a ton of duplicate or unused variables.
- Remove the check for stdint.h (always available on TR1 compilers).
- Remove the $writeheader argument from writefiles (always 1).
- Remove the following %config options:
    - DESTINATION (not used, always set to BASE).
    - EXECUTABLE (always set to inspircd).
    - OSNAME (duplicate of SYSTEM).
- Remove support for checking whether -extra modules are up to
  date (if the user is installing they won't have any -extra
  modules installed).
- Remove SSL generation messages (genssl provides these).
- Replace inappropriate uses of printf with print.
- Replace is_dir with -d from the Perl stdlib.
- Replace resolve_directory with rel2abs from the Perl stdlib.
- Rewrite getrevision to be more compact, cache the revision and
  return 'release' on non-Git builds.
- Use consistent messages for all tests.
- Use tput to clear the screen in a portable way.

Use server ids when propagating squits

m_spanningtree Change the type of TreeServer::ServerName to std::string

m_spanningtree Cleanup TreeServer and TreeSocket construction/destruction

umode_s Rewrite ProcessNoticeMasks() and remove a few related useless functions

Rewrite SnomaskManager::SendMessage() and Flush(), split code into functions

Fix generated snomask mode change string being incosistent with the input in certain cases, spotted by @Robby-

Enable the LINK snomask from m_spanningtree, remove unused FLOOD snomask

m_ojoin Stop hardcoding +Y, move global vars into a class, move initialization code into init()

Get rid of enum UserModes and mark User::modes as private

Replace hardcoded mode letters, part 3

This changes most remaining User::IsModeSet() calls to use ModeReferences for modes that were created by other modules or the core

Move SetNoticeMask(), FormatNoticeMasks() and ProcessNoticeMasks() from the User class to the snomask modehandler

Replace hardcoded mode letters, part 2

This changes all remaining Channel::IsModeSet() and Channel::GetModeParameter() calls to use ModeReferences for modes that were created by other modules or the core

Seperate ModeReference into ChanModeReference and UserModeReference

Move dynamic_reference and related classes to dynref.h

Merge pull request #589 from SaberUK/master+fix-debug-build

Fix some warnings which are causing debug builds to fail.

Fix some warnings which are causing debug builds to fail.

- Clang: private field 'module' is not used
- GCC: suggest a space before ‘;’ or explicit braces around empty
       body in ‘while’ statement

Merge pull request #587 from SaberUK/master+defer-preprocessor

Use the preprocessor to check for TCP deferring support.

Use the preprocessor to check for TCP deferring support.

m_ssl_openssl Fix inverted check that prevented certificates from being recognized as trusted

Use the correct socket related error messages on Windows

Clean up error handling in threadengine_win32

Merge pull request #588 from SaberUK/insp20+fix-socketengines

[2.0] Fix various socket engine bugs.

Merge pull request #585 from SaberUK/master+configure-engine-selection

Replace arguments for toggling socketengines  with --socketengine.

m_userip Fix incorrect "variable may be used uninitialized" warning shown by older GCC versions

Do not send too much data over SSL in one go

Some clients fail to read it entirely and the remaining data stays in their read buffer until new data arrives

Replace arguments for toggling socketengines  with --socketengine.

Fix the Poll socket engine on BSD.

Fix low risk crash when we can't determine maximum open socket count.

Remove restriction on changing <limits:maxline> while running.

Merge pull request #578 from SaberUK/master+configure-compiler-detection

Improve compiler logic in configure.

Merge pull request #573 from ShutterQuick/inspircd+sslhash

Allow the user to specify any hashing mechanism supported by the underlying SSL library

Allow the user to specify any hashing mechanism supported by the underlying SSL library

Expose compiler name to make.

- Convert ICC detection to use new variable.

Rename configuration variable to CXX to match everything else.

Correctly detect the compiler version and whether it is acceptable.

Automatically detect the compiler which the user has installed.

Purge --with-cc in favour of the CXX environment variable.

Merge pull request #577 from SaberUK/master+makefile-cleanup

Tidy up the Makefile template.

Rename CXXFLAGS to CORECXXFLAGS to respect users enviroment.

Use CXX instead of CC.

CXX is the correct variable to use as CC is the C compiler.

Add IFNEQ Makefile template macro.

- Use template macro for ICC check.
- Don't link against pthread on Darwin.

Remove 'make squeakyclean'.

This was a non-standard alias for make distclean.

Tidy up CXXFLAGS and LDFLAGS.

- Add -Iinclude at declaration instead of doing it later on.
- Add -Wextra -Wfatal-errors and -Wno-unused-parameter.
- Enable -Wall and -Wshadow on ICC.
- Remove -DPIC (useless).
- Remove empty declaration of LDFLAGS.
- Remove unneeded CXXFLAGS on Darwin.

Remove support for Interix and SunOS from the Makefile.

- SunOS was replaced with Solaris in 1992(!).
- We have a native Windows port so Interix is not needed.

Merge pull request #574 from SaberUK/master+build-comment-cleanup

Clean up unneeded build system comments.

Merge pull request #580 from SaberUK/master+restrict-network-name

Fix users being able to set <server:network> to an invalid value.

Fix users being able to set <server:network> to an invalid value.

Fix some missing newlines in the config header generator.

Fix $ModDep failing to find application-wide include files.

Remove detection of deprecated backticks which was added in 1.1b9.

Remove $Core and $Mod* comments apart from $ModDep.

Merge pull request #571 from ShutterQuick/inspircd+checkclone

Allow for skipping clone checking before DNS is complete.

Allow for skipping clone checking before DNS is complete.

Merge pull request #569 from ShutterQuick/inspircd+asrootfix

Made the launch script aware of --runasroot

Replace hardcoded mode letters passed to IsModeSet() and GetModeParameter() with ModeHandlers, part 1

Fix a few oversights

This partially reverts e0cb6bd46fb1d804ad6adea5688c844f7e1a0f6f

Made the launch script aware of --runasroot, so it does not drop privs if this is specified.

Merge pull request #563 from ElementalAlchemist/patch-1

Prevent m_cgiirc from blocking indefinitely

Prevent m_cgiirc from blocking indefinitely, resulting in registration timeouts on e.g. Mibbit when webirc is not used

m_repeat Optimize the Levenshtein() function

New version uses 2 vectors instead of a matrix
Do not shrink the vectors even if the user sets a lower <repeat:size> than before, this is because longer lines may remain in the backlog