tsoliman/scummvm — comment on commit 41ba45709d620058e478cc3485029d7cca9f9901
File:  line 0
By lordhoto:

While I can understand that a backend wants to set up additional actions, I'm currently still a bit wondering about how this is implemented.

From a naive point of view it would seem like the easiest choice here would be to allow the backend to add additional events to the global keymap itself. Maybe this is not working for any good reasons, but right now I don't see any for that. Frankly I didn't have much time to look into it though :-).

Furthermore the whole name stored in the Keymap makes this handling not that nice IMHO, the control flow looks really a bit odd. The main point behind it seems to me that we want to allow a replacement of the default global keymap. But where do we need this actually? I mean the default global keymap should only contain actions for generic use, i.e. without platform/engine specific background, shouldn't it?