[prev in list] [next in list] [prev in thread] [next in thread]
List: openbios
Subject: Re: [OpenBIOS] [PATCH] Adds get-key-map to keyboard node
From: Programmingkid <programmingkidx () gmail ! com>
Date: 2012-12-05 15:53:53
Message-ID: CF57E414-CFB6-42C8-8F50-4F8EF3A3028E () gmail ! com
[Download RAW message or body]
On Dec 4, 2012, at 4:20 PM, Mark Cave-Ayland wrote:
> On 03/12/12 21:43, Programmingkid wrote:
>
> > Would you settle for a patch that checks to see if the command key is down and \
> > also maps the control key to the command? It would mean both backwards and \
> > forward compatibility. If someone changed QEMU to detect the command key, this \
> > patch would still work unaltered.
>
> I'm not philosophically against it as long as it doesn't break compatibility for \
> anything else in the bootloader - Alex/Andreas do you know if this is the case?
After doing some research I found out detecting the command key will be difficult at \
best. The key word can't do it. The reason why the control, and shift key can be \
detected using the key word is because the control and shift key add to the value of \
the letter key being pushed. It is just a matter of knowing the range that each \
letter key will be in when pushed with a modifier key. The command key does not alter \
the value of a letter key when pushed, so detected it being pushed with the key word \
isn't an option.
The only work around that actually works with QEMU right now is to detect the control \
key as the command key. I know you don't like this idea very much, but it does \
actually work and doesn't break any thing. Also, no patch is permanent. If someone \
does figure out a way to integrate the command key into QEMU, I can always change the \
get-key-map word to use the command key.
We should stick with the get-key-map patch for now.
--
OpenBIOS http://openbios.org/
Mailinglist: http://lists.openbios.org/mailman/listinfo
Free your System - May the Forth be with you
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic