wahjam/wahjam — pull request #72 by stefanha (2012-09-14 18:08:31+00:00)
Title: Netmsg.cpp cleanups

Note that this branch is based on my wahjamsrv-qt-logging branch. GitHub will also display that commits from that branch but they are not part of this pull request.

The Net_Connection and Net_Message code has several issues that are addressed by this patch series:
1. NJClient receives no notification when the connection is refused by the server (e.g. wrong IP address or port number was given).
2. Net_Message reference counting is broken and may leak messages.
3. Some cruft has built up after converting these classes to Qt and we can clean it up.

Each patch in this series performs a single refactoring or bug fix to clean up netmsg.h/netmsg.cpp.