FMSLogo Code

Update call to WSAStartup to follow practices recommended on docs.microsoft.com
1) Version 2.2 is supported in-box since Windows XP. Previously, FMSLogo was requested 1.1, probably a legacy from the 16-bit windows machines that MSWLogo was written for.
2) The return value should be used without calling WSAGetLastError()

Refactoring: extract CServerNetworkConnection::AcceptNewConnection()
This was done to remove duplication of identical logic.

Refactoring: Stop passing a HWND as parameters in the network API.
This wasn't needed, since the only HWND is the main window.

Clear m_ConnectionIsWaitingToBeAccepted when NETACCEPTOFF is called.

Minor improvements:
* Add support for a "temporary redirect" response.
* Check for malformed HTTP Request message

Fix a problem where if a second connection was created to a NETACCEPTON server while it was already connected, then winsock would not raise any more FD_ACCEPT events and the server wouldn't be able to process any further connections.

The fix is to a set flag when NETACCEPTON is too busy to accept a second connection, then to later execute accept() once the first connection is closed.

Don't immediately return from AsyncSelect callback on error.

The important thing is to process the FD_CLOSE event even when it comes from a socket reset. Otherwise a NETACCEPTON server can get into a bad state where it no longer accepts new incoming connections.

Remove some "if (IsEnabled())" calls before calling Disable().
It is safe to call Disable() even if the connection is not enabled.

Add a comment to explain why the call to connect() is expected to fail (with WSAEWOULDBLOCK)

Don't call connect() or set up the socket if WSAAsyncSelect() fails