intel/genxml: Make assert in gen_pack_header print a message.
authorKenneth Graunke <kenneth@whitecape.org>
Mon, 7 May 2018 18:23:27 +0000 (11:23 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Tue, 8 May 2018 06:06:52 +0000 (23:06 -0700)
Python's assert can take both a condition and a string, which will cause
it to print the string if the assertion trips.  (You can't use parens as
that creates a tuple.)  Doing "condition and string" works in C, but
doesn't have the desired effect in Python.

Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
src/intel/genxml/gen_pack_header.py

index 6a4c8033a70cb8a1965bc1c2653c93ef36139a78..4bce425d12e7283d225b3600bb2ffe032743b03e 100644 (file)
@@ -220,7 +220,7 @@ def num_from_str(num_str):
     if num_str.lower().startswith('0x'):
         return int(num_str, base=16)
     else:
-        assert(not num_str.startswith('0') and 'octals numbers not allowed')
+        assert not num_str.startswith('0'), 'octals numbers not allowed'
         return int(num_str)
 
 class Field(object):