[Vm-dev] [OpenSmalltalk/opensmalltalk-vm] Remove macRoman translation of keyboard keyValue in Win32 VM (#401)

[Eliot Miranda]

> On Jun 11, 2019, at 5:04 AM, Nicolas Cellier <notifications at github.com> wrote:
> 
> There is code that transforms the presumably latin-1 key value into approximately MacRoman encoding in platforms/win32/vm/sqWin32Window.c
> I let the archeologists debate if this code dates from Jurassic period or more distant paleozoic era, all I know is that we pay for what we don't buy...
> 
> Thus I propose to entirely remove that piece of code and disentangle a bit the spaghetti keyboard handling.
> 
> We indeed do not need this extra level of complexity anymore, images do not use macRoman encoding for a long time
> 
> On the contrary, images have to undo the translation by sending macToSqueak message in various subclasses of KeyboardInputInterpreter.
> 
> For the transition period, old images running on new VMs, or new images running on old VMs, might encounter a few hickups while interpreting non ASCII characters in case of exotic KeyboardInputInterpreter. That won't be the case of main UTF32InputInterpreter since it mostly uses the utf32 code passed through eventBuffer sixth position rather than keyValue passed thru eventBuffer third, except for some CTRL+key for which the keyValue is more-than-often < 128, and thus unaffcted by the proposed change.
> 

+1


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/401#issuecomment-501342096
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/vm-dev/attachments/20190612/9cb48574/attachment.html>