fixing #3683: proper counting of TCP connections

-doxygen

-tolerate NULL

- fixed conflicting int type

- fixed rounding

- fixed conflicting int sizes

- fixed memory and coding issues

- cleaned test code

- fixed logic

- fixed uninitialised memory, logic

- fixed slow startup of service

- fixed confusion of push and pull

- check for validity of peers

- make sure there is an established channel for each peer in gossip list

- fix wrong check

- cancle own connection when peer is not in gossip list

- got rid of unneeded code

- renamed flag

- don't add own id to pull_list

- fixed behaviour of init_peer_cb()

- changed behaviour on peer disconnect

- leave peer in sampler on peer disconnect

- add additional check before peer removal

- don't put our own id in the algorithm

- restructured peer removal

- changed criteria for putting peer in pull list

- restructured check whether peer is live

- restructured in_arr() and rem_from_list()

- macro for getting/setting peer flags

-fix

-fixing misc minor bugs and style issues (unchecked return values, memory leaks, dead code

simplify logic, avoid using uninit variable

-fix callback after cancellation issue

-reduce loglevel (#3682)

-fix memory leak of address on shutdown

-fix leak on transmission error in combination with disconnect

log TCP server suspends

-nicer logging

-fix shutdown issue

generate our own HELLO before starting validations

-fix typo

-better stats for KEEPALIVEs

-better stats for KEEPALIVEs

reset blocking time for addresses upon successful connection

keep track of number of temporarily blocked addresses

-improve logging

-style

-allow UINT64_MAX without crashing

-allow UINT64_MAX without crashing

finish implementing reservation test, and fix bandwidth reservation logic

-remove unused exported symbol

add proper test for bandwidth reservation

add command to test reservations

-testcase was long dead

-testcase was long dead

-testcase was long dead

-fix badness

-ignore

-also obsolete

new testsuite, obsoleting (most of) the old one

-check for in_destroy when looking up session IDs

-dead flags

eliminate dead call from API

do not call receive before sending first message

pass correct closure

-fix NPE

-fix crash

fixing #3657 (replace ATS_Information with struct), but WIHTOUT fixing ATS testcases yet

-check RV

-fix typo

-fixing some memory leaks from #3667, also reindentation and code cleanup

-fix #3665: hello can theoretically still be NULL

fix #3664: during PS_RECONNECT_ATS, primary address may be NULL, causing an NPE if we continue here

-do not reschedule validation if in_use did not change

-just eliminate useless check, can just be false

-fix

-improve UDP logging

-eliminate dead argument, do use quota immediately

-reorder tests to have slow ones go last

-report if testcase failure is because switching is not enforced, then do not fail hard

-do not continue in case of transmission error

-minor indentation/logging fixes

do proper check for stability, in particular UINT32_MAX-0 is not a delta of 1...

perform proper locking and unlocking

-do only send HELLO to clients once on startup, do only send fully initialized HELLO to clients if we can help it, even the first one; do not perform blacklist check twice for try_connect

-tolerateNAT instant reporting

-instantly report addresses during NAT initialization if possible

-better debug output, indentation

-use backoff for reconnects

-that the tests do not catch bugs like this is a real issue...

-fix shutdown

-fix shutdown order

-fin, mention future work in TODO

-updating comments

-cleaning up bandwidth allocation routine

-when selecting addresses, kick out peers with insufficient support if we are at the limit

-further simplify

-further simplify

-further simplify

-getting there...