[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 6/6] qapi: Promote blockdev-change-medium arg
Re: [Qemu-devel] [PATCH v2 6/6] qapi: Promote blockdev-change-medium arguments to QAPI type
Fri, 20 Jan 2017 08:15:56 +0100
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)
Eric Blake <address@hidden> writes:
> On 01/19/2017 03:48 AM, Markus Armbruster wrote:
>> Eric Blake <address@hidden> writes:
>>> Having a named rather than anonymous C type will make it easier
>>> to improve the testsuite in a later patch.
>> Post it together with said later patch then.
> It was that way in the v1 series, but I had enough else going on with
> removal of dynamic JSON parsing that I thought I could pick this one
> early. But I have no problems omitting it in a v3 respin, and saving it
> for the dynamic JSON work (if we even want to revisit that discussion).
>> # @format: #optional, format to open the new image with
>> (defaults to
>> # the probed format)
>> # @read-only-mode: #optional, change the read-only mode of the device;
>> # to 'retain'
>> # Since: 2.5
>> Isn't Since: 2.5 misleading? The anonymous type goes back to 2.5, but
>> the name doesn't.
> It matches what we've done elsewhere - when refactoring .json files to
> create a new type, but where the new type doesn't represent anything
> different over the wire than what was previously sent, we've documented
> the new struct name as of the older release where the wire format was
> introduced. But as long as the command says 'since 2.5', I'm okay if we
> want to mark the struct as 'since 2.9', if that's easier to think about.
Sticking to established practice is better than inconsistent practice.
That said, I wonder why we bother to track "since" for types. "Since"
is important information for external interfaces. Why is it useful for
purely internal ones?
External visible are commands, events, and members of types used by
commands or events.
[Qemu-devel] [PATCH v2 3/6] qlist: Add convenience helpers for wrapped appends, Eric Blake, 2017/01/18
[Qemu-devel] [PATCH v2 5/6] test-qga: Actually test 0xff sync bytes, Eric Blake, 2017/01/18
[Qemu-devel] [PATCH v2 4/6] fdc-test: Avoid deprecated 'change' command, Eric Blake, 2017/01/18
[Qemu-devel] [PATCH v2 2/6] qdict: Add convenience helpers for wrapped puts, Eric Blake, 2017/01/18