summaryrefslogtreecommitdiff
path: root/lib/protocol_benchmarks.py
diff options
context:
space:
mode:
Diffstat (limited to 'lib/protocol_benchmarks.py')
-rwxr-xr-xlib/protocol_benchmarks.py12
1 files changed, 8 insertions, 4 deletions
diff --git a/lib/protocol_benchmarks.py b/lib/protocol_benchmarks.py
index b7af4f7..133e4b5 100755
--- a/lib/protocol_benchmarks.py
+++ b/lib/protocol_benchmarks.py
@@ -657,9 +657,11 @@ class MPack(DummyProtocol):
def get_serialize(self):
ret = 'serialized_size = mpack_writer_buffer_used(&writer);\n'
- ret += 'if (mpack_writer_destroy(&writer) != mpack_ok) {\n'
+ # OptionalTimingAnalysis and other wrappers only wrap lines ending with a ;
+ # We therefore deliberately do not use proper if { ... } syntax here
+ # to make sure that these two statements are timed as well.
+ ret += 'if (mpack_writer_destroy(&writer) != mpack_ok) '
ret += 'kout << "Encoding failed" << endl;\n'
- ret += '}\n'
return ret
def get_deserialize(self):
@@ -797,9 +799,11 @@ class NanoPB(DummyProtocol):
def get_deserialize(self):
ret = 'Benchmark msg = Benchmark_init_zero;\n'
ret += 'pb_istream_t stream = pb_istream_from_buffer(buf, serialized_size);\n'
- ret += 'if (pb_decode(&stream, Benchmark_fields, &msg) == false) {\n'
+ # OptionalTimingAnalysis and other wrappers only wrap lines ending with a ;
+ # We therefore deliberately do not use proper if { ... } syntax here
+ # to make sure that these two statements are timed as well.
+ ret += 'if (pb_decode(&stream, Benchmark_fields, &msg) == false) '
ret += 'kout << "deserialized failed" << endl;\n'
- ret += '}\n'
return ret
def get_decode_and_output(self):