ghaerr commented on PR #3624:
URL: https://github.com/apache/nuttx-apps/pull/3624#issuecomment-5065509365

   > Or even larger shift to something like 0xF800 as used on Microwindows.
   
   Looking ahead at Unicode, Microwindows moved its key code base to F800 
because that belongs to the Unicode BMP Private Use Area. This won't overlap 
other Unicode values if/when NuttX applications move to Unicode.
   
   > It would be problem if somebody uses the codes as index into some array 
etc... 128 or 129 shift is not so big
   
   Possibly good idea, but pushes off actively looking through source for 
problems now, and not having to change them again perhaps later. FWIW, myself 
knowing very little about NuttX, if a small shift is thought better for now, 
choosing a base of 256 at least allows space for using the upper half of 
various 256-byte multilingual code pages for non-US (e.g. European) accented 
characters, etc.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to