Makes it easier to troubleshoot keyboard issues by printing exactly
what key events the server is getting from the client.

The logic was flawed and would treat all connect errors as
if there were no addresses found.

The bug fix in b64dbf2 didn't account for the proper request
region in the case of continuous updates. Make sure we use the
proper variable for which region we've sent updates for.

A request may be for only part of the frame buffer, meaning we cannot
discard all changes just because we've send out an update. There might
still be modified areas remaining that haven't been requested yet.

Variables were reused a bit too heavily and it was possible to get
the logic at a point where the server would try to render a cursor
where it wasn't needed, and the empty update rect would cause a
crash. Clear things up by introducing some more explicit variables.

There has been some confusion if fillRect() should accept a buffer
or a pixel. This can cause misrendering if your data is not in the
native endian order. A buffer makes more sense here though, and
is what most of the callers are already assuming, so change the
API to follow that.

Some Winsock errno:s conflict with the standard ones. Try to
restrict the redefines to things relevant for sockets.

It doesn't really make sense anymore given that settings might come
from the GUI or configuration and not only the command line.

SO_REUSEADDR needs to be set before we call bind() for it to have
any effect. It got shuffled around in e4d9726.

Every reasonably current system support getaddrinfo() so get rid
of the legacy fallbacks.

It doesn't work on sockets, which require a bit more care.

It is needed by WinVNC, but got removed in 892d10a70. Also fix
a couple of issues:

 - Use getnameinfo() in order to be compatible with Windows XP
 - Make it static since it doesn't use a specific socket
 - Respect UseIPv4 and UseIPv6
 - Flags for getaddrinfo() that match binding code
 - Dummy service value for Windows compatibility

The TcpListener constructor now takes a 'struct sockaddr*' instead of
a string, and the createTcpListeners function creates TcpListener
instances for an address based on the results from getaddrinfo().

The XserverDesktop class now takes a list of TcpListener instances for
each of the RFB and HTTP sockets.

The TcpListener::closeFd member variable is not used and has been
removed.

Makes it more sane what type to use when interacting with this class.

Also avoids %lld which isn't supported on Windows.

The constants used here are unsigned so it makes more sense that
the parameters are as well.

Either by casting, or switching to a more appropriate type
for the variable.

This allows the compiler to analyse the format strings and
arguments for correctness.

The logic is a bit complex so we're going to have to silence the
warning with this explicit assignment.

The previous FIXME was incorrect. The old code did in fact continue
looking over the entire area. This commit restores that old behaviour.

It knows more details so it can track things better.

It had regressed in performance compared to 1.3. This brings it back
up to the same speed.