save_tuple(): I believe the new code for TUPLE{1,2,3} in proto 2 was
incorrect for recursive tuples. Tried to repair; seems to work OK, but
there are no checked-in tests for this yet.
diff --git a/Lib/pickle.py b/Lib/pickle.py
index 95bdd26..6f4ee5c 100644
--- a/Lib/pickle.py
+++ b/Lib/pickle.py
@@ -471,14 +471,17 @@
if proto >= 2:
for element in object:
save(element)
- # Subtle. Same as in the big comment below
+ # Subtle. Same as in the big comment below.
if id(object) in memo:
get = self.get(memo[id(object)][0])
- write(POP_MARK + get)
+ write(POP * n + get)
else:
write(_tuplesize2code[n])
+ self.memoize(object)
return
+ # proto 0, or proto 1 and tuple isn't empty, or proto > 1 and tuple
+ # has more than 3 elements.
write(MARK)
for element in object:
save(element)