summaryrefslogtreecommitdiffstats
path: root/python_modules/marshal.py
diff options
context:
space:
mode:
authorAlon Levy <alevy@redhat.com>2011-01-24 23:32:43 +0200
committerMarc-André Lureau <marcandre.lureau@gmail.com>2012-03-20 15:25:50 +0100
commit3a6de6d6fb0362df9e78fa64942a45c02bdabe53 (patch)
treec6ac424a3eec4a7c240e78d1958091877482a23d /python_modules/marshal.py
parentd6f198b3f433100a7a212be03b059876ed33e598 (diff)
downloadspice-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.py8
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()