[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] MacKeys.h: initial commit
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH] MacKeys.h: initial commit |
Date: |
Tue, 1 Mar 2016 16:19:32 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 |
On 03/01/2016 04:08 PM, Peter Maydell wrote:
> On 1 March 2016 at 22:07, Programmingkid <address@hidden> wrote:
>> This patch adds the file MacKeys.h. It is a file that contains all the
>> constants
>> for the Macintosh keyboard keycodes.
>>
>> Signed-off-by: John Arbuckle <address@hidden>
>>
>> ---
>> Note to Peter Maydell:
>> I know you wanted me to use the constants from the file HIToolbox/Events.h,
>> but
>> I can't. It only exists on Mac OS X, so other operating systems like Linux
>> would not be able to use it. This file is also used in the adb.c file, so
>> cross-platform compatibility is a must.
>
> What I suggested was that you should use the Events.h constants in
> ui/cocoa.m (which is OSX only) and have a file in include/hw/input/
> for the keys for the ADB keyboard emulation.
>
> This patch should really be in a patch series with the patches that
> use the new header.
>
>> include/hw/input/MacKeys.h | 139
>> ++++++++++++++++++++++++++++++++++++++++++++
>> 1 files changed, 139 insertions(+), 0 deletions(-)
>> create mode 100644 include/hw/input/MacKeys.h
Can we please use lower-case naming of mac-keys.h? Just because Mac
headers use Capital Names doesn't mean we have to break our internal
conventions to follow.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature