diff options
author | Fabiano FidĂȘncio <fidencio@redhat.com> | 2014-09-03 15:14:54 +0200 |
---|---|---|
committer | Christophe Fergeau <cfergeau@redhat.com> | 2014-09-18 13:13:30 +0200 |
commit | 840bcfe7a266604604a2b4a34ad68acba85ed0a7 (patch) | |
tree | 62016c48fde68f68f44c31d8c7b509ba523856d8 /python_modules | |
parent | 9385db68756f6b798e6342993d3acaf3304b363c (diff) |
python: Fix -Wsign-compare
The return of the get_array_size() is used as a limit in a loop,
being compared with an unsigned index (indexes are always unsigned).
To avoid the -Wsign-compare warning, let's cast the return of the
get_array_size() to unsigned and make GCC happier.
Diffstat (limited to 'python_modules')
-rw-r--r-- | python_modules/marshal.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/python_modules/marshal.py b/python_modules/marshal.py index 97d02e9..1eda1ba 100644 --- a/python_modules/marshal.py +++ b/python_modules/marshal.py @@ -162,11 +162,11 @@ def get_array_size(array, container_src): rows_v = container_src.get_ref(rows) # TODO: Handle multiplication overflow if bpp == 8: - return "(%s * %s)" % (width_v, rows_v) + return "(unsigned) (%s * %s)" % (width_v, rows_v) elif bpp == 1: - return "(((%s + 7) / 8 ) * %s)" % (width_v, rows_v) + return "(unsigned) (((%s + 7) / 8 ) * %s)" % (width_v, rows_v) else: - return "(((%s * %s + 7) / 8 ) * %s)" % (bpp, width_v, rows_v) + return "(unsigned) (((%s * %s + 7) / 8 ) * %s)" % (bpp, width_v, rows_v) elif array.is_bytes_length(): return container_src.get_ref(array.size[2]) else: |