diff options
author | Alon Levy <alevy@redhat.com> | 2011-01-24 23:32:43 +0200 |
---|---|---|
committer | Alon Levy <alevy@redhat.com> | 2011-01-27 12:49:11 +0200 |
commit | 3615be639bd77c2ffd340b622fc253e65d25f0b6 (patch) | |
tree | 3ec2e68246f341e8d050a397a9d62f848fac3922 /python_modules/marshal.py | |
parent | f414992e9bb8f9063f70eed8ea5f13a81632bcd3 (diff) | |
download | spice-3615be639bd77c2ffd340b622fc253e65d25f0b6.tar.gz spice-3615be639bd77c2ffd340b622fc253e65d25f0b6.tar.xz spice-3615be639bd77c2ffd340b622fc253e65d25f0b6.zip |
demarshaller/marshaller fix gcc 4.6.0
python_modules/demarshal.py and marshal.py fixes for gcc 4.6.0
warning about set but unused variables. The fixes disable creating
of variables mem_size when they are not used (demarshall) and
declaring a src variable when the message doesn't use it (marshal).
You need to touch *.proto after applying this (should add a Makefile
dependency).
Diffstat (limited to 'python_modules/marshal.py')
-rw-r--r-- | python_modules/marshal.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/python_modules/marshal.py b/python_modules/marshal.py index 9ee1466d..a010ff37 100644 --- a/python_modules/marshal.py +++ b/python_modules/marshal.py @@ -356,10 +356,12 @@ def write_message_marshaller(writer, message, is_server, private): for n in names: writer.assign("*%s_out" % n, "NULL") - src = RootMarshallingSource(None, message.c_type(), message.sizeof(), "msg") - src.reuse_scope = scope + # fix warnings about unused variables by not creating body if no members to parse + if any(x.is_fixed_nw_size() for x in message.members): + src = RootMarshallingSource(None, message.c_type(), message.sizeof(), "msg") + src.reuse_scope = scope - write_container_marshaller(writer, message, src) + write_container_marshaller(writer, message, src) writer.end_block() writer.newline() |