Message1236

Author rabidechidna
Recipients
Date 2006-09-21.07:46:35
SpamBayes Score
Marked as misclassified
Message-id
In-reply-to
Content
Logged In: YES 
user_id=1034565

I have uploaded an improved patch which additionally fixes the test_maxint64 
test case and implements a (vacuous) setter for the undocumented "fast" 
attribute of cPickle.Pickler (when this attribute is set to True in the C version 
of cPickle, the memoization is turned off - which is okay, so long as you have 
no circular references). 

This leaves 4 failures and 14 errors.

The 4 failures are from cPickle *not* raising exceptions when dumping values 
with circular references when the fast attribute is set. This is to be expected, 
since setting the fast attribute in the patched version does nothing.

4 errors (test_metaclass) come from metaclasses not working yet - the tests 
fail creating the objects to pickle, not while pickling them.

The 10 remaining NullPointerException errors are from the 
cPickleListPicklerTests test suite. This test suite tries to pass the dump 
method's bin parameter in as the Pickler constructor's file parameter. I think 
the bug is in the test suite, not in Jython.
History
Date User Action Args
2008-02-20 17:17:33adminlinkissue1561793 messages
2008-02-20 17:17:33admincreate