mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-07-27 04:13:53 -06:00
qapi: Implement deprecated-output=hide for QMP events
This policy suppresses deprecated bits in output, and thus permits "testing the future". Implement it for QMP events: suppress deprecated ones. No QMP event is deprecated right now. Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Message-Id: <20210318155519.1224118-5-armbru@redhat.com>
This commit is contained in:
parent
91fa93e516
commit
278fc2f7d3
2 changed files with 31 additions and 1 deletions
|
@ -79,6 +79,7 @@ def gen_param_var(typ: QAPISchemaObjectType) -> str:
|
|||
|
||||
def gen_event_send(name: str,
|
||||
arg_type: Optional[QAPISchemaObjectType],
|
||||
features: List[QAPISchemaFeature],
|
||||
boxed: bool,
|
||||
event_enum_name: str,
|
||||
event_emit: str) -> str:
|
||||
|
@ -107,6 +108,14 @@ def gen_event_send(name: str,
|
|||
if not boxed:
|
||||
ret += gen_param_var(arg_type)
|
||||
|
||||
if 'deprecated' in [f.name for f in features]:
|
||||
ret += mcgen('''
|
||||
|
||||
if (compat_policy.deprecated_output == COMPAT_POLICY_OUTPUT_HIDE) {
|
||||
return;
|
||||
}
|
||||
''')
|
||||
|
||||
ret += mcgen('''
|
||||
|
||||
qmp = qmp_event_build_dict("%(name)s");
|
||||
|
@ -176,6 +185,7 @@ class QAPISchemaGenEventVisitor(QAPISchemaModularCVisitor):
|
|||
#include "%(prefix)sqapi-emit-events.h"
|
||||
#include "%(events)s.h"
|
||||
#include "%(visit)s.h"
|
||||
#include "qapi/compat-policy.h"
|
||||
#include "qapi/error.h"
|
||||
#include "qapi/qmp/qdict.h"
|
||||
#include "qapi/qobject-output-visitor.h"
|
||||
|
@ -220,7 +230,7 @@ void %(event_emit)s(%(event_enum)s event, QDict *qdict);
|
|||
boxed: bool) -> None:
|
||||
with ifcontext(ifcond, self._genh, self._genc):
|
||||
self._genh.add(gen_event_send_decl(name, arg_type, boxed))
|
||||
self._genc.add(gen_event_send(name, arg_type, boxed,
|
||||
self._genc.add(gen_event_send(name, arg_type, features, boxed,
|
||||
self._event_enum_name,
|
||||
self._event_emit_name))
|
||||
# Note: we generate the enum member regardless of @ifcond, to
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue