Correct spelling in licence header.

It's licenSed and licenCe, not licenCed and licenSe.
Remove BSD makefile.

I still like the idea of having one, but it complicates things, doesn't
actually work on one of our test machines, and we can always bring it back from
the dead later.
Update Visual Studio notes a bit.
Add ConnectionPool class.

This takes on the responsibility of holding connections, as well as references
to things connections need.

It might be worth working on the interface to ConnectionPool.  The fact that it
takes a Player and a CommandHandler seems like a bit of a leaky abstraction.
Inline and remove RespondWithError.

This is used in precisely one place, and almost always will be.
Remove references to RespondIter.

This method hasn't existed in quite a while.
Clean up io_response.cpp.

Moving the response string comments to RHS, and one-lining the definition of Push.
Clean up device ID logic.

The magic number -1 is now a class const INVALID_ID.  Some logic
has been inverted to make it more obvious that INVALID_ID is a
failure return.
Change some comments in main.cpp.

Not much to say here, just that I thought the comments could be a bit terser
in places, more verbose in others, and non-existent elsewhere.
Remove non-existent functions from Playd class.

These haven't existed for months!
Fix expansion in comment.

Ta, @LordAro, for pointing this one out
Move INVALID_ID and POSITION_PERIOD to headers.

As per @LordAro's suggestion.  Avoids duplication.
Add newlines to DummyResponseSink responses.

This is closer to what real ResponseSinks do, and helps with test-cases dealing
with multiple responses coming out.
Add tests for PlayerPosition.

Woo, coverage.
Sort messages.

Fixes a todo.
Remove unused messages.

This might not be all of the unused messages, but it's a start.
Rename Connection::Close() to Depool().

This makes much more sense, and cleans up the @todo, but I'll experiment with
RAII in the next commit or so.
Clean up UvUpdateTimerCallback.

IoCore::End() is static now, so we can call it here.
Use this->tcp in Connection::Read.

This will always be the same as the old stream parameter, bugs aside.
Rework and comment IO control structures.