From c23d18a955db63723a21d1e602cc8edae0940b8e Mon Sep 17 00:00:00 2001 From: Tim Peters Date: Tue, 28 Jan 2003 01:41:51 +0000 Subject: Comments. --- Lib/pickle.py | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/Lib/pickle.py b/Lib/pickle.py index 957a87f..b6fb419 100644 --- a/Lib/pickle.py +++ b/Lib/pickle.py @@ -561,7 +561,7 @@ class Pickler: # This method does not use memoize() so that it can handle # the special case for non-binary mode. # XXX What did that comment mean? That is, what "special case for - # XXX non-binary mode? It sure *looks* like nothing special is + # XXX non-binary mode"? It sure *looks* like nothing special is # XXX happening in the INST case. memo_len = len(memo) if self.bin: @@ -699,6 +699,14 @@ class Unpickler: except _Stop, stopinst: return stopinst.value + # Return largest index k such that self.stack[k] is self.mark. + # If the stack doesn't contain a mark, eventually raises IndexError. + # This could be sped by maintaining another stack, of indices at which + # the mark appears. For that matter, the latter stack would suffice, + # and we wouldn't need to push mark objects on self.stack at all. + # Doing so is probably a good thing, though, since if the pickle is + # corrupt (or hostile) we may get a clue from finding self.mark embedded + # in unpickled objects. def marker(self): stack = self.stack mark = self.mark -- cgit v0.12