On 3/9/20 10:26 AM, Markus Armbruster wrote:
> An ALTERNATIVE's value can only be a type name. Arrays are not
> supported, yet. The text gets it right: "The form STRING is shorthand
> for { 'type': STRING }." The grammar doesn't. Fix it.
>
> Fixes: b6c37ebaaf074cac8fe8a4781dc3e79db23e914e
> Reported-by: John Snow <js...@redhat.com>
> Signed-off-by: Markus Armbruster <arm...@redhat.com>
> ---
> docs/devel/qapi-code-gen.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/docs/devel/qapi-code-gen.txt b/docs/devel/qapi-code-gen.txt
> index 59d6973e1e..e73979e182 100644
> --- a/docs/devel/qapi-code-gen.txt
> +++ b/docs/devel/qapi-code-gen.txt
> @@ -399,7 +399,7 @@ Syntax:
> 'data': ALTERNATIVES,
> '*if': COND }
> ALTERNATIVES = { ALTERNATIVE, ... }
> - ALTERNATIVE = STRING : TYPE-REF
> + ALTERNATIVE = STRING : STRING
> | STRING : { 'type': STRING, '*if': COND }
>
> Member 'alternate' names the alternate type.
>
Danke :)
Reviewed-by: John Snow <js...@redhat.com>
