Skip to content

Commit

Permalink
qapi: Replace dirty is_c_ptr() by method c_null()
Browse files Browse the repository at this point in the history
is_c_ptr() looks whether the end of the C text for the type looks like
a pointer.  Works, but is fragile.

We now have a better tool: use QAPISchemaType method c_null().  The
initializers for non-pointers become prettier: 0, false or the
enumeration constant with the value 0 instead of {0}.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Daniel P. Berrange <berrange@redhat.com>
Message-Id: <1442401589-24189-13-git-send-email-armbru@redhat.com>
  • Loading branch information
Markus Armbruster committed Sep 21, 2015
1 parent 05f43a9 commit 5710153
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 14 deletions.
16 changes: 5 additions & 11 deletions scripts/qapi-commands.py
Expand Up @@ -91,18 +91,12 @@ def gen_visitor_input_vars_decl(args):
bool has_%(argname)s = false;
''',
argname=c_name(memb.name))
if is_c_ptr(memb.type.c_type()):
ret += mcgen('''
%(argtype)s %(argname)s = NULL;
''',
argname=c_name(memb.name),
argtype=memb.type.c_type())
else:
ret += mcgen('''
%(argtype)s %(argname)s = {0};
ret += mcgen('''
%(c_type)s %(c_name)s = %(c_null)s;
''',
argname=c_name(memb.name),
argtype=memb.type.c_type())
c_name=c_name(memb.name),
c_type=memb.type.c_type(),
c_null=memb.type.c_null())

pop_indent()
return ret
Expand Down
3 changes: 0 additions & 3 deletions scripts/qapi.py
Expand Up @@ -1441,9 +1441,6 @@ def c_type(value, is_param=False):
assert isinstance(value, str) and value != ""
return c_name(value) + pointer_suffix

def is_c_ptr(value):
return value.endswith(pointer_suffix)

def genindent(count):
ret = ""
for i in range(count):
Expand Down

0 comments on commit 5710153

Please sign in to comment.