[PATCH v2] input: usb keyboard: fix CTRL+C
Trent Piepho
tpiepho at kymetacorp.com
Wed Feb 24 10:53:23 PST 2016
On Wed, 2016-02-24 at 19:12 +0100, Sascha Hauer wrote:
> > > >
> > > > - if (modstate & (1 << 0))
> > > > + if (modstate & (1 << 1))
> > > > + ascii = keycode_bb_ctrl_keys[ev->code];
ascii = keycode_bb_keys[ev->code];
ascii = ascii >= 'a' ? CTL_CH(ascii): 0;
> > Sasha, did you mean to eliminate the CTRL-keymap completely?
>
> No, I didn't, although that would be an option aswell. It would safe us
> a few more bytes in the binary. The question is, how many more keys are
> in the CTRL map that are interesting for us? Are there more than CTRL-C?
> If yes, then it's probably better to keep the map. If not, a special
> CTRL-C case in code might be the better option.
Would the above, or a slight modification, be ok? It gets almost all
the controls keys one could care about.
Ignoring control for all keys other than c doesn't make sense to me.
Does anything else do that?
More information about the barebox
mailing list