Some compilers (ehm, microsoft) returns a temporary variable for event.key.getKeyName() that is only valid during the statement and not scope.
Fix this by storing std::string instead. So the object's lifetime is the entire if statement.
When writing the X11 (linux) implementation there was a problem with X11 defining a "Display" type and we also have a Display class in the engine.
So to fix that problem and minimize the risk for running into other name conflicts. We move everything from global namespace.
