diff options
author | Alon Levy <alevy@redhat.com> | 2011-01-24 23:32:43 +0200 |
---|---|---|
committer | Marc-André Lureau <marcandre.lureau@gmail.com> | 2012-03-20 15:25:50 +0100 |
commit | 3a6de6d6fb0362df9e78fa64942a45c02bdabe53 (patch) | |
tree | c6ac424a3eec4a7c240e78d1958091877482a23d /python_modules/marshal.py | |
parent | d6f198b3f433100a7a212be03b059876ed33e598 (diff) | |
download | spice-protocol-3a6de6d6fb0362df9e78fa64942a45c02bdabe53.tar.gz spice-protocol-3a6de6d6fb0362df9e78fa64942a45c02bdabe53.tar.xz spice-protocol-3a6de6d6fb0362df9e78fa64942a45c02bdabe53.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 a82df98..5b4dfdd 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() |