Fix a code cleanup.

DHT.c:
- get_close_nodes():
    - allow two 'indirect' nodes ('indirect' as in distant from us and therefore not tested regularly, "bad")
    - be consequent when testing for NULLed results, pack nodes_list dense
    - (logging) dump number of found entries from assoc
- returnedip_ports():
    - fix code cleanup, the entry to be added is about the node we were told, not about the node who told us

assoc.c:
- Assoc_get_close_entries(): break from loops as soon as a marking-invalid-node is hit
Merge pull request #678 from FullName/retipp-fix

Fix a code cleanup.
added groupchat actions
small fix
Fixed comment.
Code cleanups.

No functionality changed.
Astyled the code.
Added simple test for tox.
Added new tox autotest.