Beefed up the tests by putting in more "for proto in protocols:" outer
loops. Renamed DATA and BINDATA to DATA0 and DATA1. Included
disassemblies, but noted why we can't test them. Added XXX comment to
cPickle about a mysterious comment, where pickle and cPickle diverge
in how they number PUT indices.
diff --git a/Modules/cPickle.c b/Modules/cPickle.c
index 00e6829..c284056 100644
--- a/Modules/cPickle.c
+++ b/Modules/cPickle.c
@@ -719,6 +719,11 @@
goto finally;
/* Make sure memo keys are positive! */
+ /* XXX Why?
+ * XXX And does "positive" really mean non-negative?
+ * XXX pickle.py starts with PUT index 0, not 1. This makes for
+ * XXX gratuitous differences between the pickling modules.
+ */
p++;
if (!( py_ob_id = PyLong_FromVoidPtr(ob)))