[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 8/9] qapi: Factor out compat_policy_input_ok()
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 8/9] qapi: Factor out compat_policy_input_ok() |
Date: |
Fri, 29 Oct 2021 19:01:58 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0 |
On 10/29/21 18:55, Markus Armbruster wrote:
> Markus Armbruster <armbru@redhat.com> writes:
>
>> The code to check policy for handling deprecated input is triplicated.
>> Factor it out into compat_policy_input_ok() before I mess with it in
>> the next commit.
>>
>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
>> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> ---
>> include/qapi/compat-policy.h | 7 +++++
>> qapi/qapi-visit-core.c | 18 +++++--------
>> qapi/qmp-dispatch.c | 51 +++++++++++++++++++++++++++---------
>> qapi/qobject-input-visitor.c | 19 +++-----------
>> 4 files changed, 55 insertions(+), 40 deletions(-)
>> +bool compat_policy_input_ok(unsigned special_features,
>> + const CompatPolicy *policy,
>> + ErrorClass error_class,
>> + const char *kind, const char *name,
>> + Error **errp)
>> +{
>> + if ((special_features & 1u << QAPI_DEPRECATED)
>> + && !compat_policy_input_ok1("Deprecated",
>> + policy->deprecated_input,
>> + error_class, kind, name, errp)) {
>> + return false;
>> + }
>> + return true;
>> +}
>> +
>> Visitor *qobject_input_visitor_new_qmp(QObject *obj)
>> {
>> Visitor *v = qobject_input_visitor_new(obj);
>
> I'm moving the new functions along with @compat_policy to qapi-util.c,
> so the QAPI visitors survive linking without qmp-dispatch.o.
OK. I am waiting your series to get in before respining my QAPI
sysemu/user series which will re-scramble meson.build here.
Probably too late for this release. Not super important, it just
saves energy avoiding generating / building unused files.
- [PATCH v2 2/9] qapi: Mark unstable QMP parts with feature 'unstable', (continued)