|
From: | Eric Blake |
Subject: | Re: [PATCH 1/2] Introduce (x86) CPU model deprecation API |
Date: | Thu, 4 Jun 2020 06:59:53 -0500 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.8.0 |
On 6/4/20 3:07 AM, Robert Hoo wrote:
+++ b/qapi/machine-target.json @@ -309,7 +309,8 @@ 'static': 'bool', '*unavailable-features': [ 'str' ], 'typename': 'str', - '*alias-of' : 'str' }, + '*alias-of' : 'str', + 'deprecated' : 'bool' },Missing documentation of the new member. Should it be optional (present only when true)?Which document do you mean?
A few lines earlier is '@alias-of: ...'; you'll need to add a similar line for '@deprecated', mentioning it is '(since 5.1)'.
How to make it optional?
Name it '*deprecated', then deal with 'has_deprecated' in the C code for the cases where the member should be output.
-- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org
[Prev in Thread] | Current Thread | [Next in Thread] |