Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-31829: Make protocol 0 pickles be loadable in text mode in Python 2. #11859

Merged
merged 2 commits into from
May 31, 2019

Conversation

serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Feb 14, 2019

Escape \r, \0 and \x1a (end-of-file on Windows) in Unicode strings.

https://bugs.python.org/issue31829

Escape ``\r``, ``\0`` and ``\x1a`` (end-of-file on Windows) in Unicode strings.
Co-Authored-By: serhiy-storchaka <storchaka@gmail.com>
@serhiy-storchaka serhiy-storchaka merged commit 38ab7d4 into python:master May 31, 2019
@miss-islington
Copy link
Contributor

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@serhiy-storchaka serhiy-storchaka deleted the pickle-binary branch May 31, 2019 08:29
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 31, 2019
… 2. (pythonGH-11859)

Escape ``\r``, ``\0`` and ``\x1a`` (end-of-file on Windows) in Unicode strings.
(cherry picked from commit 38ab7d4)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot
Copy link

GH-13693 is a backport of this pull request to the 3.7 branch.

miss-islington added a commit that referenced this pull request Jun 12, 2019
… 2. (GH-11859)

Escape ``\r``, ``\0`` and ``\x1a`` (end-of-file on Windows) in Unicode strings.
(cherry picked from commit 38ab7d4)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
… 2. (pythonGH-11859)

Escape ``\r``, ``\0`` and ``\x1a`` (end-of-file on Windows) in Unicode strings.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants