[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.5 v2] input: Document why x-input-send-eve
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH for-2.5 v2] input: Document why x-input-send-event is still experimental |
Date: |
Fri, 13 Nov 2015 10:05:59 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Gerd Hoffmann <address@hidden> writes:
> On Do, 2015-11-12 at 11:50 -0700, Eric Blake wrote:
>> The x-input-send-event command was introduced in 2.2 with mention
>> that it is experimental, but now that several releases have elapsed
>> without any changes, it would be nice to document why that was done
>> and should still remain experimental in 2.5.
>>
>> Meanwhile, our documentation states that we prefer 'lower-case',
>> rather than 'CamelCase', for qapi enum values. The InputButton and
>> InputAxis enums violate this convention. However, because they are
>> currently used primarily for generating code that is used internally;
>> and their only exposure through QMP is via the experimental
>> 'x-input-send-event' command, we are free to change their spelling.
>> Of course, it would be nicer to delay such a change until the same
>> time we promote the command to non-experimental. Adding
>> documentation will help us remember to do that rename.
>>
>> We have plans to tighten the qapi generator to flag instances of
>> inconsistent use of naming conventions; if that lands first, it
>> will just need to whitelist these exceptions until the time we
>> settle on the final interface.
>
> Reviewed-by: Gerd Hoffmann <address@hidden>
Applied to qapi-next for 2.5, thanks!