This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author alexandre.vassalotti
Recipients alexandre.vassalotti, amaury.forgeotdarc, belopolsky, jnoller
Date 2010-06-28.21:16:39
SpamBayes Score 0.0016740318
Marked as misclassified No
Message-id <1277759801.42.0.784770504779.issue3385@psf.upfronthosting.co.za>
In-reply-to
Content
> Do I understand correctly that the issue is that python
> Pickler class has dispatch attribute but C Pickler does
> not?

Yes.

> The add_dispatch_check-0.patch patch does not seem
> to add class attribute, it adds an instance attribute
> instead.

I know. That's why I said it was a preliminary patch. :-) Also, see msg71159 about the class vs instance attribute issue.

There's a lot of code out there that do crazy things with pickle's dispatch dictionary. For now, it is best if we leave the method names alone. If people wants to keep doing their funky stuff with pickle, then we can point them to pickle._Pickler and pickle._Unpickler, which will always point to the Python implementations.

That said, I do not believe it's a good idea to add the dispatch attribute to _pickle anymore. But, I acknowledge there is a need for overriding the pickling mechanism for specific types. We should find a clean way to support this.

I am closing this issue as "won't fix".
History
Date User Action Args
2010-06-28 21:16:41alexandre.vassalottisetrecipients: + alexandre.vassalotti, amaury.forgeotdarc, belopolsky, jnoller
2010-06-28 21:16:41alexandre.vassalottisetmessageid: <1277759801.42.0.784770504779.issue3385@psf.upfronthosting.co.za>
2010-06-28 21:16:40alexandre.vassalottilinkissue3385 messages
2010-06-28 21:16:39alexandre.vassalotticreate