Author eli.bendersky
Recipients Arfrever, danielsh, einarfd, eli.bendersky, ezio.melotti, flox, georg.brandl, jcea, larry, python-dev, santoso.wijaya
Date 2013-01-01.00:32:50
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1357000370.87.0.162837799779.issue16076@psf.upfronthosting.co.za>
In-reply-to
Content
A couple of high-level questions about the C code (this isn't a detailed review yet):

1. Why did you choose to implement __reduce__ and not __getstate__?
2. A lot of code appears to be shared between element_setstate_from_attributes and the init method implementation, can it be refactored?
3. I've been thinking about the C<->Python pickle compatibility, and it may be too much trouble at this point for no very strong benefit. What could make more sense at some point is implementing a __getstate__/__setstate__ in both the Python and C versions that are compatible (i.e. produce the same things). But this may mean changing the way the Python Element is pickled in 3.3, which may not be acceptable. So we may prefer to forego the compatibility in 3.3 and then for 3.4 make the change. I'm still not sure what is best here.
History
Date User Action Args
2013-01-01 00:32:50eli.benderskysetrecipients: + eli.bendersky, georg.brandl, jcea, larry, ezio.melotti, Arfrever, flox, santoso.wijaya, python-dev, einarfd, danielsh
2013-01-01 00:32:50eli.benderskysetmessageid: <1357000370.87.0.162837799779.issue16076@psf.upfronthosting.co.za>
2013-01-01 00:32:50eli.benderskylinkissue16076 messages
2013-01-01 00:32:50eli.benderskycreate