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.

classification
Title: Keyword argument support in cPickle
Type: Stage:
Components: Extension Modules Versions:
process
Status: closed Resolution: accepted
Dependencies: Superseder:
Assigned To: Nosy List: ddorfman, loewis
Priority: normal Keywords: patch

Created on 2004-07-22 08:25 by ddorfman, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
cpickle.diff ddorfman, 2004-07-22 08:25 cPickle change and test case
Messages (5)
msg46417 - (view) Author: Dima Dorfman (ddorfman) Date: 2004-07-22 08:25
Add keyword argument support to the "dump", "dumps", and 
"Pickler"
callables in the cPickle module. This provides better API
compatibility with the pure-Python pickle module. There are 
several
other differences, but this one is probably the most user-visible 
and
likely to be noticed. And it's easy to fix! The callables chosen are
those that take multiple arguments.

This patch addresses bug #888594
msg46418 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2004-07-26 12:16
Logged In: YES 
user_id=21627

The patch is incorrect: the first argument is called object.
Please also double-check consistency with the documentation.

I'm going to apply the revised version for 2.4 only, as this
is a subtle behaviour change. If you think it should be
backported to 2.3 as well, please indicate so.
msg46419 - (view) Author: Dima Dorfman (ddorfman) Date: 2004-07-26 13:18
Logged In: YES 
user_id=908995

The patch does what it claims to do--be more compatible with the 
pure-Python pickle module, which uses "obj" as the first argument. I noticed 
that when I made the change and should have mentioned it in the initial 
comment. I intend to submit a patch to the documentation to change the first 
argument to "obj". Although I think "object" is better, someone might be 
relying on the pure-Python using obj even though the documentation is wrong 
about that.

I see no reason to backport this to 2.3.
msg46420 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2004-07-27 05:23
Logged In: YES 
user_id=21627

I see. Thanks for the patch; I have applied it as 

pickletester.py 1.57
NEWS 1.1056
cPickle.c 2.151
msg46421 - (view) Author: Dima Dorfman (ddorfman) Date: 2004-07-28 09:22
Logged In: YES 
user_id=908995

Thanks Martin. I submitted the documentation update as SF #999280, 
and this patch addressed bug #888594, which can now be closed.
History
Date User Action Args
2022-04-11 14:56:05adminsetgithub: 40617
2004-07-22 08:25:20ddorfmancreate