qemu-devel
[Top][All Lists]
Advanced

[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!



reply via email to

[Prev in Thread] Current Thread [Next in Thread]