[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/6] qapi: AllocVisitor
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/6] qapi: AllocVisitor |
Date: |
Wed, 17 Jun 2015 15:42:36 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
"Kővágó Zoltán" <address@hidden> writes:
> 2015-06-17 09:56 keltezéssel, Markus Armbruster írta:
>> "Kővágó, Zoltán" <address@hidden> writes:
>>
>>> Simple visitor that recursively allocates structures with only optional
>>> variables. Unions are initialized to the first type specified. Other non
>>> optional types are not supported.
>>
>> Sounds dubious :)
>>
>> Can you explain why it's useful? I guess later patches provide an
>> example.
>
> Oh, crap, ignore this commit. Just realized that in the refactorings I
> did I removed all references to this visitor... so it's not needed.
Glad I asked before reviewing the code :)
- Re: [Qemu-devel] [PATCH v2 1/6] qapi: qapi for audio backends, (continued)
- Re: [Qemu-devel] [PATCH v2 1/6] qapi: qapi for audio backends, Markus Armbruster, 2015/06/17
- Re: [Qemu-devel] [PATCH v2 1/6] qapi: qapi for audio backends, Kővágó Zoltán, 2015/06/17
- Re: [Qemu-devel] [PATCH v2 1/6] qapi: qapi for audio backends, Markus Armbruster, 2015/06/17
- Re: [Qemu-devel] [PATCH v2 1/6] qapi: qapi for audio backends, Kővágó Zoltán, 2015/06/17
- Re: [Qemu-devel] [PATCH v2 1/6] qapi: qapi for audio backends, Markus Armbruster, 2015/06/17
- Re: [Qemu-devel] [PATCH v2 1/6] qapi: qapi for audio backends, Kővágó Zoltán, 2015/06/17
- Re: [Qemu-devel] [PATCH v2 1/6] qapi: qapi for audio backends, Eric Blake, 2015/06/17
[Qemu-devel] [PATCH v2 4/6] qapi: AllocVisitor, Kővágó, Zoltán, 2015/06/16
[Qemu-devel] [PATCH v2 5/6] audio: use qapi AudioFormat instead of audfmt_e, Kővágó, Zoltán, 2015/06/16
[Qemu-devel] [PATCH v2 6/6] audio: -audiodev command line option, Kővágó, Zoltán, 2015/06/16