Hehe. Oops :P
That comment was *meant* to errr... try and catch you out? FWIW I did a quick test without it and it seemed to work fine

(Is this something we should write a unit test for, since there are lots of edge cases? If not, maybe we should at least put in the comments to the methods things like:
* Xterm is dodgy, as is RemoteDesktop
* Some apps only work with key codes and not the unicode string
* etc.
)