summaryrefslogtreecommitdiffstats
path: root/python_modules/marshal.py
diff options
context:
space:
mode:
authorAlon Levy <alevy@redhat.com>2011-01-24 23:32:43 +0200
committerAlon Levy <alevy@redhat.com>2011-01-27 12:49:11 +0200
commit3615be639bd77c2ffd340b622fc253e65d25f0b6 (patch)
tree3ec2e68246f341e8d050a397a9d62f848fac3922 /python_modules/marshal.py
parentf414992e9bb8f9063f70eed8ea5f13a81632bcd3 (diff)
downloadspice-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.py8
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()