From 12424bc0eff82e0ebf5134cca1139b7d4b6faed0 Mon Sep 17 00:00:00 2001 From: Skip Montanaro Date: Thu, 23 May 2002 01:43:05 +0000 Subject: force gzip module to open files using 'b'inary mode. closes patch #536278. --- Lib/gzip.py | 4 ++++ Lib/test/test_gzip.py | 6 +++++- 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/Lib/gzip.py b/Lib/gzip.py index b2bbeda..9e198c7 100644 --- a/Lib/gzip.py +++ b/Lib/gzip.py @@ -35,6 +35,10 @@ class GzipFile: def __init__(self, filename=None, mode=None, compresslevel=9, fileobj=None): + # guarantee the file is opened in binary mode on platforms + # that care about that sort of thing + if mode and 'b' not in mode: + mode += 'b' if fileobj is None: fileobj = self.myfileobj = __builtin__.open(filename, mode or 'rb') if filename is None: diff --git a/Lib/test/test_gzip.py b/Lib/test/test_gzip.py index a5660a9..d42dee6 100644 --- a/Lib/test/test_gzip.py +++ b/Lib/test/test_gzip.py @@ -18,7 +18,7 @@ data2 = """/* zlibmodule.c -- gzip-compatible data compression */ f = gzip.GzipFile(filename, 'wb') ; f.write(data1 * 50) ; f.close() -f = gzip.GzipFile(filename, 'rb') ; d = f.read() ; f.close() +f = gzip.GzipFile(filename, 'r') ; d = f.read() ; f.close() verify(d == data1*50) # Append to the previous file @@ -75,4 +75,8 @@ for pos in range(0, 256, 16): f.write('GZ\n') f.close() +f = gzip.GzipFile(filename, 'r') +verify(f.myfileobj.mode == 'rb') +f.close() + os.unlink(filename) -- cgit v0.12