[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 06/28] qapi: Add some union tests
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v5 06/28] qapi: Add some union tests |
Date: |
Thu, 26 Mar 2015 16:58:47 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 03/26/2015 07:23 AM, Markus Armbruster wrote:
>> One more:
>>
>> [...]
>>> diff --git a/tests/qapi-schema/alternate-conflict-string.json
>>> b/tests/qapi-schema/alternate-conflict-string.json
>>> new file mode 100644
>>> index 0000000..5fd1a47
>>> --- /dev/null
>>> +++ b/tests/qapi-schema/alternate-conflict-string.json
>>> @@ -0,0 +1,8 @@
>>> +# FIXME: we should reject anonymous unions with multiple
>>> string-like branches
>>> +{ 'enum': 'Enum',
>>> + 'data': [ 'hello', 'world' ] }
>>> +{ 'union': 'MyUnion',
>>> + 'discriminator': {},
>>> + 'data': { 'one': 'str',
>>> + 'two': 'Enum' } }
>>> +
>>
>> /home/armbru/work/qemu/.git/rebase-apply/patch:325: new blank line at EOF.
>
> Huh. I thought I had git set up to reject me from making commits like
> that locally, but obviously not.
There's another one in PATCH 13:
/home/armbru/work/qemu/.git/rebase-apply/patch:156: new blank line at EOF.
> http://wiki.qemu.org/Contribute/SubmitAPatch should probably mention the
> magic one-time setup to use to turn this type of checking on...
Feel free to add it :)
- Re: [Qemu-devel] [PATCH v5 06/28] qapi: Add some union tests, (continued)
- Re: [Qemu-devel] [PATCH v5 06/28] qapi: Add some union tests, Kevin Wolf, 2015/03/31
- Re: [Qemu-devel] [PATCH v5 06/28] qapi: Add some union tests, Eric Blake, 2015/03/31
- Re: [Qemu-devel] [PATCH v5 06/28] qapi: Add some union tests, Eric Blake, 2015/03/31
- Re: [Qemu-devel] [PATCH v5 06/28] qapi: Add some union tests, Kevin Wolf, 2015/03/31
- Re: [Qemu-devel] [PATCH v5 06/28] qapi: Add some union tests, Markus Armbruster, 2015/03/31
Re: [Qemu-devel] [PATCH v5 06/28] qapi: Add some union tests, Markus Armbruster, 2015/03/26
[Qemu-devel] [PATCH v5 26/28] qapi: Drop inline nested type in query-version, Eric Blake, 2015/03/24
[Qemu-devel] [PATCH v5 22/28] qapi: Whitelist commands that don't return dictionary, Eric Blake, 2015/03/24