microsoft / node-native-keymap

Provide OS keyboard layout functionality as a nodejs module
MIT License
136 stars 37 forks source link

Clear keyboard buffer after ToUnicode call (Fixes #1 and #4) #5

Closed the-ress closed 8 years ago

the-ress commented 8 years ago

Clear keyboard buffer after calling ToUnicode.

Adapted from: http://archives.miloush.net/michkap/archive/2006/04/06/569632.html http://archives.miloush.net/michkap/archive/2007/10/27/5717859.html

msftclas commented 8 years ago

Hi @the-ress, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution! This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

alexdima commented 8 years ago

@the-ress Thank you very much! A great catch, I was unaware that the keyboard buffer needed to be cleared!

the-ress commented 8 years ago

I didn't know about it either until I dug into this issue. (That has been annoying me for months.)