[kbd] [PATCH] src/libkeymap: add support for parsing more unicode values

[Alexey Gladkov]

On Mon, Mar 01, 2021 at 03:49:52PM +0100, Anisse Astier wrote:
> On Mon, Mar 01, 2021 at 03:09:39PM +0100, Alexey Gladkov wrote:
> > On Sat, Feb 27, 2021 at 03:36:11PM +0100, Anisse Astier wrote:
> > > The auto-generated (with ckbcomp) file fr-bepo_afnor did not load (even
> > > partially), because of an U+1f12f (copyleft symbol) that is wrongly
> > > parsed, generating this error message:
> > > 
> > > 	too many (160) entries on one line
> > > 
> > > Fix libkeymap so that the keymap can be parsed, even if the offending
> > > character won't be loaded because of the ushort limitation of the
> > > kb_value KDSKBENT uapi.
> > > 
> > > It's better to have the keymap partially loaded than not at all.
> > 
> > Nop. Partially keymap loading is very dangerous. You can get a completely
> > unusable console. The libkeymap shouldn't break the console if it is known
> 
> By partially, I meant that only a single character of a single key
> wouldn't load. I'm curious, what would be the implications here ? How
> could it break ?

In this particular case, nothing will happen. An unicode character will
not be loaded and the user will probably see a warning. And this approach
is the best way to do it.

> I tried loading this keymap, and didn't see any averse effect. The
> character was missing, and that's about it. Why wouldn't we want that ?