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...

-simplify flow, redistribute bandwidth after adding address

do not force disconnect just because client does not request address anymore

-simplify flow

-simplify flow

-simplify logic

-useless assertion removal

-indent

changing s_get API to return void instead of address, have plugin use existing mechanism to signal address activation

-simplify logic, convert breaks into asserts

-restructuring, adding comments

-remove fprintf

eliminate super-lazy functions

clean up find_quality_property_index and get_active_address_it

cleaning up get_active_address

cleaning up address_decrement

check for nan

removing dead argument

avoid several linear scans and allocations by combining AddressWrapper and asi structs

use new 'get_connectivity' API which is also RC-counting, instead of another map

expose connectivity desire of clients to ATS plugins

adding FIXME about 'struct Session'

-clarify API

cleaning up preference_peers as well once they are no longer needed

cleaning up gnunet-service-ats_preferences.c

cleaning up gnunet-service-ats_preferences.c

-adding comments, minor cleanup start

-fix config, shutdown issue

-rename for naming consistency

fix testcase FTBFS

simplify normalization logic, also have clients access norm array of address directly

more dead and duplicate code elimination

-updating gettext m4 macros

-bringing copyright tags up to FSF standard

-bringing copyright tags up to FSF standard

DCE

deduplicate code for quota parsing

deduplicate code for quota parsing

- change in POTFILES.in

distinct samplers for client and Brahms protocol

minor cleanup

simplify logic: just use string directly for the plugin name

use lower-case

simplify logic

simplify logic

-sane timeout

test ATS tests to pass again

-fix ril delete address handling

-fix misc minor crashes

-fixing silly NPEs

get rid of stupid network[i]=i array

-use only one nc per client

-fix segv

first pass at cleaning up ATS plugin API

starting with cleaning up gnunet-service-ats_preferences.c

starting with cleaning up gnunet-service-ats_preferences.c

dead code elimination

dead code elimination

avoid void *, use proper type for client

-misc cleanups

cleaning up gnunet-service-ats_normalization.c

- fixed resource leakages

- trying to fix coverity defects

- change in POTFILES.in

-move header

separating out feedback from preferences

-cleaning up gnunet-service-ats_performance-*

-cleaning up gnunet-service-ats_connectivity-*

-cleaning up gnunet-service-ats_addresses.h

-cleaning up gnunet-service-ats_addresses.c

big ATS refactoring, no serious semantic changes should stem from this

Various changes:

- removed "in_use" from being passed to ATS, no
  longer needed as we tell ATS when we cannot
  follow a suggestion by removing the address;
  simplifies the code a lot
- better handling of state machine, fallback to
  primary session if address switch to new
  session timed out
- nicer logging
- more comments
- new testcase failures, because of
  #3652, #3651 and #3650 (or more that i missed).

improved test description