We need to ensure the inter-thread messaging queues are empty (more
specifically, not locked) before destroying them. The best way to ensure
that is to explicitly wait for thread termination.

Controller::WasKeyReleasedThisFrame() is now keyboard->key_released() so
there is usually no need for a controller. They will completely disappear
when joystick axis binding has been ported to InputDevice.

Most input and GUI files are now in a common ui/ directory. The platform/
directory is no longer, as everythyng was moved to application/ or ui/.

In particular, get rid of the enum magic; we can use map<string,int> if we
really need some named enums. Which we usually don’t.

I was arguing that Enter and other keys should be part of the text
input API, but it’s hard to decide what Tab actually does. I prefer
to give more freedom to the client application.

We were no longer calling Render() on primitives with a null source.