aboutsummaryrefslogtreecommitdiffstats
path: root/src/intel/genxml
diff options
context:
space:
mode:
authorMathieu Bridon <[email protected]>2018-06-17 14:40:31 +0200
committerEric Engestrom <[email protected]>2018-08-01 14:26:19 +0100
commita71df20855e1d5e875042ca34de9401951fb96a6 (patch)
treee605d1b00c5a3cb8cb00747309fca84db79e50d9 /src/intel/genxml
parent8678fe537a5eeb9a537fc540672bf375e802c004 (diff)
python: Explicitly use byte strings
In both Python 2 and 3, zlib.Compress.compress() takes a byte string, and returns a byte string as well. In Python 2, the script was working because: 1. string literalls were byte strings; 2. opening a file in unicode mode, reading from it, then passing the unicode string to compress() would automatically encode to a byte string; On Python 3, the above two points are not valid any more, so: 1. zlib.Compress.compress() refuses the passed unicode string; 2. compressed_data, defined as an empty unicode string literal, can't be concatenated with the byte string returned by compress(); This commit fixes this by explicitly using byte strings where appropriate, so that the script works on both Python 2 and 3. Signed-off-by: Mathieu Bridon <[email protected]> Reviewed-by: Eric Engestrom <[email protected]>
Diffstat (limited to 'src/intel/genxml')
-rw-r--r--src/intel/genxml/gen_zipped_file.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/intel/genxml/gen_zipped_file.py b/src/intel/genxml/gen_zipped_file.py
index 1d6bd568246..616409183f0 100644
--- a/src/intel/genxml/gen_zipped_file.py
+++ b/src/intel/genxml/gen_zipped_file.py
@@ -42,10 +42,10 @@ def main():
print("} genxml_files_table[] = {")
xml_offset = 0
- compressed_data = ''
+ compressed_data = b''
for i in range(1, len(sys.argv)):
filename = sys.argv[i]
- xml = open(filename).read()
+ xml = open(filename, "rb").read()
xml_length = len(xml)
root = et.fromstring(xml)