- extban_usermode.c: minor cleanup/dedupe

- Allow 'hideidle' and 'hidechans' to be set in 'general::oper_only_umodes'

- Treat 0 minute klines as permanent ones;  rename valid_tkline() to valid_aline_time();  remove TK_SECONDS/TK_MINUTES cruft

- conf.c:attach_iline(): just let the client silently in if their class is full and they are exempted from limits. No point
  in explicitely telling them that their class is full and they have exceed_limit = yes

- match.c: use bool in some places

- Change MAX_INCLUDE_DEPTH to 5

- channel_mode.c:send_mode_changes_client(), send_mode_changes_server(): don't send the appending space in the TMODE/MODE message in case paracount == 0

- Change accept/callerid related code to no longer re-use the 'struct split_nuh_item' and have
  its own 'struct AcceptItem' instead. Minor other cleanups to callerid related functions.

- conf_lexer.l: port over C comment handling we introduced to hopm a while ago

- conf.c: move class_ip_limit_add() from conf_attach() to attach_iline() where all the other session limit tests take place.
  Moving this is safe as it is guaranteed that a auth conf can never be attached twice.

- Valid characters allowed in user names for local clients are now [A-Za-z0-9._-]

- user.c:introduce_client(): use sendto_server() to actually benefit from shared data buffers

- res.c: remove timeout_resolver() wrapper;  remove unused 'next_time' calculation in timeout_query_list() as this function is run each second anyway

- Final removal of backwards compatibility code for ircd-hybrid 8.2.22 and below. Also,
  minimum supported version of Anope is now 2.0.8.

- whowas.h, whowas.c: rename list/hash dlink_node members of the Whowas struct to actually make sense

- Stylistical changes

- monitor.c:monitor_del_from_hash_table(): replace dlinkFind/dlinkDelete combo with dlinkFindDelete

- extban.c:extban_get_isupport(): remove useless post-increment on 'p'

- send.c:sendto_channel_butone(), sendto_anywhere(): test for IsDead()
  before actually calling send_message()

- Implemented IRCv3.2 `echo-message` extension

- isupport.c:isupport_init(): add BOT=B to RPL_ISUPPORT as proposed in https://github.com/ircv3/ircv3-ideas/issues/43

User mode `B` has been implemented. Clients with that mode set are marked as a bot
in both `WHOIS` and `WHO`. This mode can only be set by IRC operators (as long as
the `bot` directive is set in `general::oper_only_umodes`), servers, and services.

This mode can for example be used to mark HOPM as official network bot.

- monitor.c:monitor_count_memory(): add +1 for '\0'

- user.c:register_local_user(): reduce buffer size from 512 to (sizeof("Invalid username []") + sizeof(client->username))

- conf_parser.y: as of bison 3.5, an explicit POSIX yacc compliance warning is thrown when %type isn't applied to non-terminals

- Rebuild autogenerated files

- channel.c:channel_send_members(): replace IRCD_BUFSIZE with sizeof(buf)

- ircd.c: tweak event_try_connections interval. try_connections() gets called every 15 seconds.

- For consistency, the `general::network_desc` configuration directive has been renamed to `general::network_description`

- Remove serno.h

- id.c:valid_sid(), valid_uid(): don't allow lowercase letters in server/user IDs. Changes provided by Adam.

- conf.c, user.c: "Too many connections", "auth {} block is full", and "Too many clients, rejecting" server
  notices now no longer go to user modes u/f, but go to user mode j (rejection notices) instead

- channel.c:check_spambot_warning(): move assignment

- Remove CAPAB_SVS

- m_stats.c, listener.c: listener->active is of type bool. Use true/false where appropriate;  change inetport() to bool

- Stylistic changes

- Style corrections; reformatting

- conf.c:operator_find(): allow matching against client->realhost;  deduplicate

- Make use of listener_has_flag() in various places;  client-only listeners are now marked with a 'C' in "/stats P"

- Added 'client' option to listener::flags

- user.c:introduce_client(): remove old backwards compatibility code for non rhost servers

- Removed MAXIMUM_LINKS_DEFAULT

- packet.c:parse_client_queued(): use 'return' consistently. 'break' needs to be used only in case of a state change

- tls_gnutls.c, tls_openssl.c, tls_wolfssl.c: fixed ret vs. retlen vs. length inconsistency

- channel.c:channel_send_namereply(): remove show_eon argument. In hybrid we no longer have a case where a names reply needs to be sent with no RPL_ENDOFNAMES

- Extban $t of type 'acting' has been implemented. This extban allows matching based on TLS protocol version and/or cipher suite

- Rename find_channel_link() to member_find_link()

- Move remove_a_mode() from m_join.c to channel.c and rename it to channel_demote_members();  have m_sjoin.c make use of it as well

- s_bsd.c:comm_checktimeouts(): minor cleanups; remove pointless tests on timeout handlers

- Replace remaining occurrences of IsMember with find_channel_link and remove IsMember entirely

- Fixed remaining style issues

- Remove hardcoded numeric digit from RPL_WHOISCHANNELS

- parse.c:parse(): move comment to where it really belongs to

- Rename Client::certfp to Client::tls_certfp

- parse.c:parse_handle_numeric(): unconfuse some extra smart looking condition

- Style corrections

- Refactor Message flags

- Each type of command handler now has its own min/max argument count pair;  remove remaining argument checks from all modules

- user.c:register_local_user(): remove assert() on client->servptr that's now pointless

- channel_mode.c:add_id(): actually store the raw ban mask in ban->banstr for servers/remote clients (fixes github issue #18)

- channel_invite.c:invite_find(): add some basic documentation

- Replace some occurrences of sprintf with snprintf

- Removed 'spoof_notice' from auth::flags. Spoofs are not 'hard spoofs' anymore as they used to be back in the days.
  The real host can always be seen via /whois. Also with vhosts now this notice doesn't make much sense anyway

- ircd.c:setup_fdlimit(): fixed setrlimit() throwing EINVAL on macOS/catalina (#16 on github)

- Remove unused header includes

- Add src/extban_nick.c

- tls_wolfssl.c: make use of recently added wolfSSL_CTX_set1_groups_list if available. Kudos to the guys at wolfSSL for implementing it that fast.

- autoreconf

- user.c:introduce_client(): rename 'server_p' to 'server'

- motd.c:motd_lookup(): check against client->realhost, too

- Rename most occurrences of 'client_p' to 'client'

- extban_server.c: allow $s to be used for invite exceptions

- extban.c:extban_get_isupport(): remove extraneous cast

- Re-create autogenerated files

- ircd.c: remove unused header inclusion

- Renamings:
    tls_new_cred -> tls_new_credentials
    tls_free_cred -> tls_free_credentials
    tls_verify_cert -> tls_verify_certificate

- conf.c:conf_read_files(): remove second buffer which doesn't make any sense here

- tls_gnutls.c:tls_new_cred(): verify gnutls_global_init() run

- conf_parser.y: style corrections; mostly whitespace changes

- Stylistic changes

- Move ax_with_tls.m4 to ax_arg_with_tls.m4

- autoreconf

- tls_gnutls.c, tls_openssl.c: tls_get_cipher, tls_get_version: reduce buffer sizes

- tls_openssl.c:tls_new_cred(): (re)set default ciphers if there aren't any defined in the configuration

- tls_openssl.c:tls_new_cred(): stylistic changes

- tls_openssl.c:tls_get_cipher(): drop the explicit bits used information as this is redundant

- In the serverinfo {} block, the following configuration directives have been renamed:

   ssl_certificate_file -> tls_certificate_file
   ssl_dh_param_file -> tls_dh_param_file
   ssl_dh_elliptic_curve -> tls_supported_groups
   ssl_cipher_list -> tls_cipher_list
   ssl_message_digest_algorithm -> tls_message_digest_algorithm

- In the operator {} block, the following configuration directives have been renamed:

   ssl_certificate_fingerprint -> tls_certificate_fingerprint
   ssl_connection_required -> tls_connection_required
   
- In the connect {} block, the following configuration directives have been renamed:

   ssl_cipher_list -> tls_cipher_list
   ssl_certificate_fingerprint -> tls_certificate_fingerprint

- Rebuilt parser files

- tls_gnutls.c, tls_openssl.c: stylistic changes

- tls_openssl.c: further modernization: replace SSL_CTX_set_tmp_ecdh with SSL_CTX_set1_groups_list

- tls_openssl.c:tls_new_cred(): fixed compile warning

- OpenSSL now runs with automatic curve selection if no curve is defined in serverinfo::ssl_dh_elliptic_curve

- conf.c:conf_dns_callback(): kill a memcpy()

- Rename several functions in conf.c as follows:
    set_default_conf -> conf_set_defaults
    read_conf -> conf_read
    validate_conf -> conf_validate
    lookup_confhost -> conf_resolve_host
    check_client -> conf_check_client
    read_conf_files -> conf_read_files
    clear_out_old_conf -> conf_clear

- user.c: merge check_xline() into register_local_user()

- tls_gnutls.c:tls_verify_cert(): we're actually not interested in the length of the certificate list. i.e. pass NULL as second argument to gnutls_certificate_get_peers().

- The connection timeout for connect{} blocks can now be configured via the 'connect::timeout' configuration directive

- Various stylistic changes

- Rename struct Membership to ChannelMember
- Rename Channel::last_knock to last_knock_time
- Rename Channel::last_invite to last_invite_time
- Rename chptr to channel

- Command handlers are now of type void again

- tls_gnutls.c:tls_new_cred(): fixed segfault issue in case there's no DH parameter file. Reported by Dominic Hargreaves <dom@debian.org>

- s_bsd.c:setup_socket(): enable TCP_QUICKACK

- hostmask.c:try_parse_v4_netmask(): made 'addb' of type uint8_t

- conf_class.c:class_find(): fixed logic that wouldn't allow to pick up inactive classes leading to '/rehash' not updating existing classes and rather creating duplicates

- s_bsd.c:comm_connect_tcp(): kill another memcpy()

- hostmask.c:try_parse_v6_netmask(): change 'unsigned short' to 'uint16_t'

- hostmask.c:try_parse_v4_netmask(): make UBSan shut up

- listener.c:inetport(): kill pointless copying of listener->addr

- Kill a few memcpy() not needed in some places

- send.c:send_queued_all(): deduplicate

- send.c: stylistic changes

- client.c:exit_client(): add assert()

- client.c, server.c: use monotonic time for server connection cycle

- Move oper_privs_as_string() from conf.c to m_stats.c

- Removed general::tkline_expire_notices configuration directive and added user mode +X (sees *LINE expiration notices) instead

- tls_gnutls.c: avoid maintaining multiple priority strings

- send.c:send_queued_write(): refactoring

- packet.c:read_packet(): read until EAGAIN for non TLS clients, too

- send.c: split up if statements

- Fixed issue where timeouts set with comm_setselect() were not working as in most places this function has been called with seconds when it really expected
  milliseconds. Removed all related multiply/divide by 1000 nonsense that's sprinkled in the code as ircd currently does support one-second granularity anyway.

- auth.c: change ident timeouts to 4/4

- ircd.c:main(): improve seed generation

- Should be using monotonic clock almost everywhere now; sort out mixture and misuse of 'firsttime', 'lasttime' and 'since' in the Connection structure

- event.c:event_time_set(): fix compile warning

- channel.c:check_spambot_warning(): remove extraneous parentheses

- Add support for monotonic clocks and have some timers make use of it;  remove event_set_back_events()

- auth.c:auth_connect_callback(): allow a read timeout of 3 seconds

- auth.c: remove auth_timeout_queries() and related code. Just let the dns and identd timeouts finish the request
- Removed IDENTTIMEOUT from '/set' options. We now no longer allow users to change this on runtime. identd timeout is 5 seconds now

- auth.c:auth_dns_callback(): replace `EmptyString()` test with just `namelength == 0`

- auth.c:auth_dns_callback(): minor readability improvements

- server.c:server_connect_callback(): remove redundant memcpy(). Address is already being copied to client->ip in server_connect()

- Recreate parser files

- server.c:server_connect(): cleanup address binding related mess

- The 'serverinfo::vhost' and 'serverinfo:vhost6' configuration directives have been deprecated. If you need to bind() a specific address you can specify one in the connect {} block
- The 'connect::vhost' configuration directive has been renamed to 'connect::bind'

- config_parser.y: minor style corrections

- conf_lexer.l: remove unused 'passwd'

- client.c:check_conf_klines(): skip dead clients on the unknown_list, too

- Fixed bug where handshaking servers wouldn't timeout

- conf_connect_allowed, find_conf_by_address, find_address_conf, find_dline_conf: drop the aftype argument

- tls_gnutls.c, tls_openssl.c: disable TLSv1.1

- send.c:sendto_match_servs(): add IsDead test

- auth.c: sprinkle some assert()

- Tidy up some enums;  misc style corrections

- Made Connection::oper_warn_count_down unsigned; clean up some logic in check_spambot_warning()

- Recreate autogenerated files

- Recreate autogenerated files

- tls_gnutls.c, tls_openssl.c: disable TLSv1.0

- fdlist.c:fd_note(): don't leak memory on F->desc if it is already allocated

- Recreate autogenerated files

- autoreconf

- autoreconf

- conf.c:verify_access(): improve readability

- Sprinkle some assert()

- Get rid of Connection::allow_read and have parse_client_queued() determine max lines to be processed

- server_die(): remove die/restart enum. Use bool instead.

- Stylistic changes

- More bool conversion. This should be everything now.

- Fix Coverity CID 1475602:  Memory - illegal accesses  (UNINIT)

- Further parse_aline() cleanups; get rid of find_user_host()
- /KLINE and /DLINE now no longer supports nick names

- Cleanup parse_aline() related code

- Make use of bool even more

- Make use of the bool data type in some more places

- Use bool data type in struct Whowas

- Cleanups to add_id and del_id; have these function take a dlink_list pointer to the channel's banmask list

- Use a boolean type for chan_mode::only_servers and chan_mode::only_opers

- patricia.c:ascii2prefix(): fixed compile warning

- patricia.c:patricia_clear(): remove commented out xfree(). For this we have patricia_destroy()

- patricia.c:patricia_clear(): must set patricia->head to NULL

- class_ip_limit_add, class_ip_limit_remove: use void pointers for 'addr'

- conf_db.c:open_db_read(): simplify

- client_svstag.c:svstag_attach(): documentation

- patricia.c: stylistic changes

- patricia.c:ascii2prefix(): Fix CID 1474567:  Memory - illegal accesses  (RETURN_LOCAL). Using "string", which points to an out-of-scope variable "save".

- Makefile.am: add patricia.c to ircd_SOURCES

- patricia: add patricia_make_and_lookup_addr()

- Sprinkle some assert(); cleanup existing ones

- channel.c:channel_do_join(): deduplicate

- s_bsd.c:ssl_handshake: use 'F' instead of 'client_p->connection->fd'

- s_bsd_epoll.c: get rid of the fixed size ep_fdlist and add support for a growing epoll_event array. Some code derived from libevent.

- s_bsd_epoll.c:comm_setselect(): use memset 0 on 'ep_event'

- user.c: style corrections

- Style corrections

- AUTHORS, version.c: purge a bunch of yada yada yada

- Style corrections

- log.c: further cleanups

- Killed Connection::aftype. Use Client::ip.ss.ss_family instead.

- Killed userhost.c. Rewrote everything to use ipcache.c

- propset

- conf_gecos.c, conf_resv.c: stylistic changes

- Implemented channel mode 'N' which prevents users from changing their nick while in a channel with that mode set

- conf_pseudo.c:pseudo_message_handler(): stylistic changes

- Move pseudot_stats() to m_stats.c

- conf.c:clear_out_old_conf(): free ConfigServerHide.hidden_name

- Add channel_get_list() to channel.c which should be used when accessing channel_list outside of channel.c

- conf_gecos.c: remove unused header include

- Stylistic changes

- Remove CHW and HOPS server capabilities. These are mandatory for quite some time now.

- ircd_signal.c:setup_signals(): use NULL instead of 0 as third argument for sigaction()

- client.c:client_free(): remove some assert() that are now no longer valid

- list.c: revert 'dlinkDelete(): reset m->data as well'

- Move close_connection() from s_bsd.c to client.c

- hash.c:free_list_task(): properly unlink nodes from their lists even though memory is freed solely for debugging purposes

- Rip out mempool

- Move fd_dump() to m_stats.c; rename it to stats_fdlist()

- Fixed bug spotted by Erick J. Velez where ircd would not remove RPL_WHOISOPERATOR based svstags when deoppering.

- "JOIN 0" is now no longer supported

- res.c:restart_resolver(): Added sanity test. ResolverFileDescriptor might be NULL at this point

- autoreconf

- Style corrections

- auth.c:make_auth(): properly assign 'auth' to 'auth->client->connection->auth'

- s_bsd_kqueue.c:comm_select(): fixed build

- client.c:client_free(): fixed assert() broken by previous commit

- autoreconf

- The memory pooling allocator can be disabled with the --disable-mempool switch now

- autoreconf

- client.c:client_free(): set client_p->connection = NULL after freeing it

- s_bsd.c: removed useless auth.h header include

- auth.h, auth.c: made release_auth_client() 'static'; removed declaration in auth.h

- auth.c:release_auth_client(): use the AddFlag macro

- client.c:exit_client(): added assert(); style correction

- list.c:dlinkDelete(): reset m->data as well

- client.c:client_free(): sprinkle some assert()

- ircd.c:main(): show 'ircd_version' and 'serno' in 'Server ready' message

- channel.c:check_spambot_warning(): stylistic changes; improve readability

- motd.c:motd_lookup(): fixed indentation

- Change RPL_WELCOME to rfc2812 style now that we have the realhost information in Client::realhost

- channel_mode.c:set_channel_mode(): use explicit NULL

- channel_mode.c:channel_modes(): fixed invalid test

- auth.c: add documentation

- Store real host information in Client.realhost and extend the UID message to send the actual host.
  This allows operators to see the real host of a client in /whois and /whowas.

- server_capab.h, server_capab.c: propset

- autoreconf

- irc_string.c: sync strlcpy() and strlcat() with latest versions from openbsd

- Update Riedel's email address

- Fixed NULL pointer dereference in chm_nosuch()

- Get rid of chm_registered() and chm_simple_oper()

- Add RPL_ETRACEEND numeric taken from irc2.11

- user.c:show_lusers(): get rid of redundant tests

- numeric.c: add channel modes 'Lu' to RPL_MYINFO

- Rebuilt configuration parser files

- Add chptr->mode.mode manipulation macros

- Implement channel mode 'u' which hides bmasks (+b/+e/+I) to non-chanops everywhere

- send.c: replace IsDefunct() test in sendto_wallops_flags(), and sendto_realops_flags() with just IsDead()

- send.c:sendto_realops_flags(): initialize 'ntype' with "???"

- channel.c: fixed style inconsistencies

- channel_mode.c:send_mode_changes_client(): remove senseless code

- channel_mode.c: send_mode_changes_(client|server): remove length test on parabuf; minor other cleanups

- channel_mode.c: fixed possible issue in send_mode_changes_(client|server) where it wouldn't strip the trailing space on TMODE with more than MAXMODEPARAMS parameters

- Fixed old bug where lusers/stats u would report an invalid max local connection count in RPL_STATSCONN

- Rename get_client_name() to client_get_name()

- server.c:server_hunt(): use 'tmp' instead of 'node->data'

- numeric.c: made ERR_SSLONLYCHAN more descriptive as requested by r0d3nt

- Cleanup some bad mess in mo_trace(). This also fixes some bug where we would get RPL_TRACELINK replies with
  "ac2ptr_is_NULL!!" from remote servers including SIDs or UIDs.

- Get rid of Count.myserver

- No longer put servers on the 'global_client_list'. This was just bad.

- Rename some functions in client.c to comply with naming convention

- Rename hunt_server() to server_hunt()

- 'opers_bypass_callerid' now defaults to 'yes'

- send.c:sendto_channel_butone(): split long if statement

- misc.c: add some comments

- conf_class.c:class_init(): make things look saner

- Update copyright years

- channel.c:channel_do_join(): rename variable

- patricia.c: whitespace change

- Make hash.c:free_list_task() use dlinkDelete() instead of dlinkFindDelete()

- Import FLOODTIME changes from p4

- aptricia.c, patricia.h: add patricia_try_search_exact_addr() and patricia_try_search_best_addr()

- patricia.c: white-space changes

- patricia.c, patricia.h:New_Patricia() fixed compile warning; removed useless 0 assignments

- patricia.c: cleanup New_Prefix2()

- patricia.c, patricia.h: rewrite prefix_toa() to suck less

- patricia.c:Deref_Prefix(): removed pointless assert(); stylistic changes

- patricia.c, patricia.h: indenting; remove extraneous parentheses

- patricia.c: remove 'num_active_patricia' not needed

- patricia.h, patricia.c: cleanup prefix_toa() to fit our needs; make it use snprintf()

- patricia.h, patricia.c: we don't need these functions outside patricia.c

- patricia.h, patricia.c: removed void_fn_t typedef

- patricia.c:try_search_exact(): add missing NULL return

- patricia.c:try_search_best(): make it work with both ipv4 and ipv6; add pointer tests as ascii2prefix() can return NULL

- Minor style corrections and constifications

- Implement INVITE expirations

- conf.c:conf_error_report(), yyerror(): use const

- conf.c:conf_error_report(): fixed compile warning

- Rebuilt parser files

- Added 'channel::max_invites' configuration directive

- Get rid of clear_invites_channel() and clear_invites_client(). Use clear_invite_list() instead.

- conf.c: fixed a lineno off-by-one for parser warnings in included configuration files

- conf_lexer.l:cinclude(): we don't use yyin

- Show GnuTLS/OpenSSL library/header versions in /INFO as suggested by Adam

- version.c: update metalrock's email

- Add Andrew Strutt to AUTHORS and version.c

- conf_pseudo.h, conf_pseudo.c: added pseudo_get_list() for later use

- Fixed more svn properties as pointed out by jackal

- Fixed svn properties

- list.c: fixed typo in comments

- user.c:register_remote_user(): remove TS5 leftover

- Change userhost.c to deal with ip addresses only. Also we no longer care about usernames/ident replies.
  Due to the hash function in hash.c not ideal for ip addresses, we'll be using either iphash.c, or
  patricia.c soon for this.

- s_user.c: rename variables for consistency

- Minor cleanup to send_umode() to get rid of the redundant struct Client * pointer

- res.c: reslist::id is now of unsigned type; remove reslist::sent which we currently don't need

- Merge send_capabilities() and show_capabilities() into get_capabilities()

- token_vector(): use unsigned where appropriate

- hash.c:free_list_task(): we really want to use dlinkFindDelete() here instead of just dlinkDelete()

- user.c:register_local_user(): style correction

- Rename init_netio() to netio_init()

- Rewrite invite handling to be more flexible for future additions such as client based invite throttling, invite expiration, etc

- channel.c:add_invite(): trim down client's invite list until there are no more
  than ConfigChannel.max_channels entries on the list