From: Guido van Rossum <guido@python.org>
Date: Fri, 5 Feb 1999 20:55:16 +0000 (+0000)
Subject: Should open files in binary mode!
X-Git-Tag: v1.5.2b2~125
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=2013ba48a68fea36373e3c21be76083be4c774a0;p=thirdparty%2FPython%2Fcpython.git

Should open files in binary mode!

Also finally get rid of some obsolete commented-out access statements.

A note about the previous checkin: I believe it's correct, but I found
something strange: the file Lib/test/audiotest.au in the Python
distribution was evidently encoded in u-LAW format but had its
encoding set to 2, i.e. linear-8.  I hope that this is a mistake
caused by some conversion program that produced this .au file; I just
found it on a website.
---

diff --git a/Lib/sunau.py b/Lib/sunau.py
index 9fee78c689fb..b49a5dafae50 100644
--- a/Lib/sunau.py
+++ b/Lib/sunau.py
@@ -148,14 +148,11 @@ def _write_u32(file, x):
 		file.write(chr(int(data[i])))
 
 class Au_read:
-## 	access _file, _soundpos, _hdr_size, _data_size, _encoding, \
-## 		  _sampwidth, _framesize, _framerate, _nchannels, \
-## 		  _info: private
 
 	def __init__(self, f):
 		if type(f) == type(''):
 			import __builtin__
-			f = __builtin__.open(f, 'r')
+			f = __builtin__.open(f, 'rb')
 		self.initfp(f)
 
 	def __del__(self):
@@ -280,14 +277,11 @@ class Au_read:
 		self._file = None
 
 class Au_write:
-## 	access _file, _framerate, _nchannels, _sampwidth, _framesize, \
-## 		  _nframes, _nframeswritten, _datawritten, _info, \
-## 		  _comptype: private
 
 	def __init__(self, f):
 		if type(f) == type(''):
 			import __builtin__
-			f = __builtin__.open(f, 'w')
+			f = __builtin__.open(f, 'wb')
 		self.initfp(f)
 
 	def __del__(self):
@@ -410,7 +404,6 @@ class Au_write:
 	#
 	# private methods
 	#
-## 	if 0: access *: private
 
 	def _ensure_header_written(self):
 		if not self._nframeswritten: