From 2b5ff073ab9c232307f82dfd1cab0589ef293df5 Mon Sep 17 00:00:00 2001 From: Guido van Rossum Date: Sun, 15 Apr 2001 13:32:27 +0000 Subject: [PATCH] Get rid of the seek() method on the _Mailbox class. This was a cut-and-paste copy of the seek() method on the _Subfile class, but it didn't make one bit of sense: it sets self.pos, which is not used in this class or its subclasses, and it uses self.start and self.stop, which aren't defined on this class or its subclasses. This is purely my own fault -- I added this in rev 1.4 and apparently never tried to use it. Since it's not documented, and of very questionable use given that there's no tell(), I'm ripping it out. This resolves SF bug 416199 by Andrew Dalke: mailbox.py seek problems. --- Lib/mailbox.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/Lib/mailbox.py b/Lib/mailbox.py index 2f96106c6e8..29322ed16be 100755 --- a/Lib/mailbox.py +++ b/Lib/mailbox.py @@ -14,14 +14,6 @@ class _Mailbox: self.seekp = 0 self.factory = factory - def seek(self, pos, whence=0): - if whence==1: # Relative to current position - self.pos = self.pos + pos - if whence==2: # Relative to file's end - self.pos = self.stop + pos - else: # Default - absolute position - self.pos = self.start + pos - def next(self): while 1: self.fp.seek(self.seekp)