[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v8 25/40] qapi: Require valid names

From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH v8 25/40] qapi: Require valid names
Date: Mon, 04 May 2015 19:43:41 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)

Eric Blake <address@hidden> writes:

> Previous commits demonstrated that the generator overlooked various
> bad naming situations:
> - types, commands, and events need a valid name
> - enum members must be valid names, when combined with prefix
> - union and alternate branches cannot be marked optional
> Valid upstream names match [a-zA-Z][a-zA-Z0-9_-]*; valid downstream
> names match __[a-zA-Z][a-zA-Z0-9._-]*.  Enumerations match the
> weaker [a-zA-Z0-9._-]+ (in part thanks to QKeyCode picking an enum
> that starts with a digit, which we can't change now due to
> backwards compatibility).  Rather than call out three separate
> regex, this patch just uses a broader combination that allows both
> upstream and downstream names, as well as a small hack that
> realizes that any enum name is merely a suffix to an already valid
> name prefix (that is, any enum name is valid if prepending _ fits
> the normal rules).
> We could reject new enumeration names beginning with a digit by
> whitelisting existing exceptions.  We could also be stricter
> about the distinction between upstream names (no leading
> underscore, no use of dot) and downstream (mandatory leading
> double underscore), but it is probably not worth the bother.
> Signed-off-by: Eric Blake <address@hidden>

Reviewed-by: Markus Armbruster <address@hidden>

reply via email to

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