[issue37502] Fix default argument handling for buffers argument in pickle.loads

[Antoine Pitrou]

Change by Antoine Pitrou :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37502] Fix default argument handling for buffers argument in pickle.loads

[miss-islington]

miss-islington  added the comment:


New changeset 25cb4fd4fb0f44d2b6bf38379634f3d22b77aa17 by Miss Islington (bot) 
in branch '3.8':
bpo-37502: handle default parameter for buffers argument of pickle.loads 
correctly (GH-14593)
https://github.com/python/cpython/commit/25cb4fd4fb0f44d2b6bf38379634f3d22b77aa17


--
nosy: +miss-islington

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37502] Fix default argument handling for buffers argument in pickle.loads

[Antoine Pitrou]

Antoine Pitrou  added the comment:


New changeset 898318b53d921298d1f1fcfa0f415844afbeb318 by Antoine Pitrou 
(Markus Mohrhard) in branch 'master':
bpo-37502: handle default parameter for buffers argument of pickle.loads 
correctly (GH-14593)
https://github.com/python/cpython/commit/898318b53d921298d1f1fcfa0f415844afbeb318


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37502] Fix default argument handling for buffers argument in pickle.loads

[miss-islington]

Change by miss-islington :


--
pull_requests: +14718
pull_request: https://github.com/python/cpython/pull/14948

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37502] Fix default argument handling for buffers argument in pickle.loads

[Kyle Stanley]

Kyle Stanley  added the comment:

Thanks for reporting the issue. On PR-14593 I reviewed the changes with 
interpreter screenshots showing the output of "pickle.loads(pickle.dumps(1, 
protocol=pickle.HIGHEST_PROTOCOL), buffers=None)" in the latest version of 
cpython master compared to the branch from the PR.

--
nosy: +aeros167

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37502] Fix default argument handling for buffers argument in pickle.loads

[Markus Mohrhard]

Markus Mohrhard  added the comment:

Sorr, I somehow managed to overwrite my title before submitting with the one 
from one if the results of my searches.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue37502] Fix default argument handling for buffers argument in pickle.loads

[Markus Mohrhard]

Change by Markus Mohrhard :


--
title: Pure Python pickle module should not depend on _pickle.PickleBuffer -> 
Fix default argument handling for buffers argument in pickle.loads

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com