Use __init__ instead of init.

Also use CDDB_PATH and CDDB_WRITE_DIR environment variables in cddb.py.

Lib/irix5/cddb.py

@@ -4,8 +4,8 @@
 # Usage is as follows:
 #
 # import readcd
-# r = readcd.Readcd().init()
-# c = Cddb().init(r.gettrackinfo())
+# r = readcd.Readcd()
+# c = Cddb(r.gettrackinfo())
 #
 # Now you can use c.artist, c.title and c.track[trackno] (where trackno
 # starts at 1).  When the CD is not recognized, all values will be the empty
@@ -55,7 +55,7 @@ def tochash(toc):
 	return hash
 	
 class Cddb:
-	def init(self, tracklist):
+	def __init__(self, tracklist):
 		if posix.environ.has_key('CDDB_PATH'):
 			path = posix.environ['CDDB_PATH']
 			cddb_path = string.splitfields(path, ',')
@@ -104,7 +104,7 @@ class Cddb:
 			except IOError:
 				pass
 		if not hasattr(self, 'file'):
-			return self
+			return
 		import regex
 		reg = regex.compile('^\\([^.]*\\)\\.\\([^:]*\\):\t\\(.*\\)')
 		while 1:
@@ -138,14 +138,17 @@ class Cddb:
 					continue
 				self.track[trackno] = value
 		f.close()
-		return self
 
 	def write(self):
+		import posixpath
 		if posix.environ.has_key('CDDB_WRITE_DIR'):
 			dir = posix.environ['CDDB_WRITE_DIR']
 		else:
 			dir = posix.environ['HOME'] + '/' + _cddbrc
 		file = dir + '/' + self.id + '.rdb'
+		if posixpath.exists(file):
+			# make backup copy
+			posix.rename(file, file + '~')
 		f = open(file, 'w')
 		f.write('album.title:\t' + self.title + '\n')
 		f.write('album.artist:\t' + self.artist + '\n')

Lib/irix5/cdplayer.py

@@ -4,8 +4,8 @@
 # Usage is as follows:
 #
 # import readcd
-# r = readcd.Readcd().init()
-# c = Cdplayer().init(r.gettrackinfo())
+# r = readcd.Readcd()
+# c = Cdplayer(r.gettrackinfo())
 #
 # Now you can use c.artist, c.title and c.track[trackno] (where trackno
 # starts at 1).  When the CD is not recognized, all values will be the empty
@@ -17,7 +17,7 @@
 cdplayerrc = '.cdplayerrc'
 
 class Cdplayer:
-	def init(self, tracklist):
+	def __init__(self, tracklist):
 		import string
 		self.artist = ''
 		self.title = ''
@@ -38,7 +38,7 @@ class Cdplayer:
 			import posix
 			f = open(posix.environ['HOME'] + '/' + cdplayerrc, 'r')
 		except IOError:
-			return self
+			return
 		import regex
 		reg = regex.compile('^\\([^:]*\\):\t\\(.*\\)')
 		s = self.id + '.'
@@ -62,7 +62,6 @@ class Cdplayer:
 					trackno = string.atoi(name[6:])
 					self.track[trackno] = value
 		f.close()
-		return self
 
 	def write(self):
 		import posix

Lib/irix5/readcd.doc

@@ -18,7 +18,7 @@ read and the callbacks will be called.
 Initialization.
 ===============
 
-r = readcd.Readcd().init([cd-player [, mode]])
+r = readcd.Readcd([cd-player [, mode]])
 
 The optional arguments are the name of the CD device and the mode.
 When "mode" is not specified, it defaults to 'r' (which is the only

Lib/irix5/readcd.py

@@ -22,7 +22,7 @@ def _dopnum(self, cb_type, data):
 		func(arg, cb_type, data)
 
 class Readcd:
-	def init(self, *arg):
+	def __init__(self, *arg):
 		if len(arg) == 0:
 			self.player = cd.open()
 		elif len(arg) == 1:
@@ -38,7 +38,6 @@ class Readcd:
 		self.end = 0
 		self.status = None
 		self.trackinfo = None
-		return self
 
 	def eject(self):
 		self.player.eject()

Lib/plat-irix5/cddb.py

@@ -4,8 +4,8 @@
 # Usage is as follows:
 #
 # import readcd
-# r = readcd.Readcd().init()
-# c = Cddb().init(r.gettrackinfo())
+# r = readcd.Readcd()
+# c = Cddb(r.gettrackinfo())
 #
 # Now you can use c.artist, c.title and c.track[trackno] (where trackno
 # starts at 1).  When the CD is not recognized, all values will be the empty
@@ -55,7 +55,7 @@ def tochash(toc):
 	return hash
 	
 class Cddb:
-	def init(self, tracklist):
+	def __init__(self, tracklist):
 		if posix.environ.has_key('CDDB_PATH'):
 			path = posix.environ['CDDB_PATH']
 			cddb_path = string.splitfields(path, ',')
@@ -104,7 +104,7 @@ class Cddb:
 			except IOError:
 				pass
 		if not hasattr(self, 'file'):
-			return self
+			return
 		import regex
 		reg = regex.compile('^\\([^.]*\\)\\.\\([^:]*\\):\t\\(.*\\)')
 		while 1:
@@ -138,14 +138,17 @@ class Cddb:
 					continue
 				self.track[trackno] = value
 		f.close()
-		return self
 
 	def write(self):
+		import posixpath
 		if posix.environ.has_key('CDDB_WRITE_DIR'):
 			dir = posix.environ['CDDB_WRITE_DIR']
 		else:
 			dir = posix.environ['HOME'] + '/' + _cddbrc
 		file = dir + '/' + self.id + '.rdb'
+		if posixpath.exists(file):
+			# make backup copy
+			posix.rename(file, file + '~')
 		f = open(file, 'w')
 		f.write('album.title:\t' + self.title + '\n')
 		f.write('album.artist:\t' + self.artist + '\n')

Lib/plat-irix5/cdplayer.py

@@ -4,8 +4,8 @@
 # Usage is as follows:
 #
 # import readcd
-# r = readcd.Readcd().init()
-# c = Cdplayer().init(r.gettrackinfo())
+# r = readcd.Readcd()
+# c = Cdplayer(r.gettrackinfo())
 #
 # Now you can use c.artist, c.title and c.track[trackno] (where trackno
 # starts at 1).  When the CD is not recognized, all values will be the empty
@@ -17,7 +17,7 @@
 cdplayerrc = '.cdplayerrc'
 
 class Cdplayer:
-	def init(self, tracklist):
+	def __init__(self, tracklist):
 		import string
 		self.artist = ''
 		self.title = ''
@@ -38,7 +38,7 @@ class Cdplayer:
 			import posix
 			f = open(posix.environ['HOME'] + '/' + cdplayerrc, 'r')
 		except IOError:
-			return self
+			return
 		import regex
 		reg = regex.compile('^\\([^:]*\\):\t\\(.*\\)')
 		s = self.id + '.'
@@ -62,7 +62,6 @@ class Cdplayer:
 					trackno = string.atoi(name[6:])
 					self.track[trackno] = value
 		f.close()
-		return self
 
 	def write(self):
 		import posix

Lib/plat-irix5/readcd.doc

@@ -18,7 +18,7 @@ read and the callbacks will be called.
 Initialization.
 ===============
 
-r = readcd.Readcd().init([cd-player [, mode]])
+r = readcd.Readcd([cd-player [, mode]])
 
 The optional arguments are the name of the CD device and the mode.
 When "mode" is not specified, it defaults to 'r' (which is the only

Lib/plat-irix5/readcd.py

@@ -22,7 +22,7 @@ def _dopnum(self, cb_type, data):
 		func(arg, cb_type, data)
 
 class Readcd:
-	def init(self, *arg):
+	def __init__(self, *arg):
 		if len(arg) == 0:
 			self.player = cd.open()
 		elif len(arg) == 1:
@@ -38,7 +38,6 @@ class Readcd:
 		self.end = 0
 		self.status = None
 		self.trackinfo = None
-		return self
 
 	def eject(self):
 		self.player.eject()