➜
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.
Created on 2014-08-27 10:50 by ncoghlan, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| backslashreplace_decode.patch | serhiy.storchaka, 2014-09-23 12:27 | review | ||
| backslashreplace_decode_2.patch | serhiy.storchaka, 2014-11-25 12:43 | review | ||
| Messages (9) | |||
|---|---|---|---|
| msg225966 - (view) | Author: Alyssa Coghlan (ncoghlan) *  |
Date: 2014-08-27 10:50 | |
In the discussion on issue 18814, Antoine pointed out that in a Python 3 world, using backslashescape during decoding actually makes sense - it lets you accurately report arbitrary bytes in the sequence, without needing surrogateescape or surrogatepass to be used when encoding later. |
|||
| msg227352 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2014-09-23 12:27 | |
Here is a patch. |
|||
| msg231655 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2014-11-25 12:43 | |
Synchronized with the tip (resolved conflicts with issue22470 and issue19676). |
|||
| msg234144 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2015-01-16 19:44 | |
Could you make a review Nick to get this feature in the first alpha. |
|||
| msg234607 - (view) | Author: Alyssa Coghlan (ncoghlan) * |
Date: 2015-01-24 12:57 | |
+1 from me for merging, although I suspect you'll need to adjust the codecs.rst changes first. |
|||
| msg234691 - (view) | Author: Roundup Robot (python-dev)  |
Date: 2015-01-25 20:57 | |
New changeset dd8a03e98158 by Serhiy Storchaka in branch 'default': Issue #22286: The "backslashreplace" error handlers now works with https://hg.python.org/cpython/rev/dd8a03e98158 |
|||
| msg234694 - (view) | Author: R. David Murray (r.david.murray) * |
Date: 2015-01-25 21:24 | |
Looks like the buildbots aren't happy with this change. |
|||
| msg234706 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2015-01-25 23:27 | |
Yes, I see. The patch exposed existing bug in decoding error handing. See issue23321 for this. |
|||
| msg234784 - (view) | Author: Roundup Robot (python-dev) |
Date: 2015-01-26 22:27 | |
New changeset 1e8937861ee3 by Victor Stinner in branch 'default': Issue #22286, #23321: Fix failing test on Windows code page 932 https://hg.python.org/cpython/rev/1e8937861ee3 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:07 | admin | set | github: 66482 |
| 2015-01-27 08:54:56 | serhiy.storchaka | set | status: open -> closed resolution: fixed stage: commit review -> resolved |
| 2015-01-26 22:27:25 | python-dev | set | messages: + msg234784 |
| 2015-01-25 23:27:12 | serhiy.storchaka | set | messages: + msg234706 |
| 2015-01-25 23:18:27 | steve.dower | set | nosy:
+ steve.dower |
| 2015-01-25 21:24:52 | r.david.murray | set | nosy:
+ r.david.murray messages: + msg234694 |
| 2015-01-25 20:57:47 | python-dev | set | nosy:
+ python-dev messages: + msg234691 |
| 2015-01-24 12:57:20 | ncoghlan | set | messages:
+ msg234607 stage: patch review -> commit review |
| 2015-01-16 19:44:00 | serhiy.storchaka | set | messages: + msg234144 |
| 2015-01-12 23:03:41 | martin.panter | set | nosy:
+ martin.panter |
| 2014-11-25 12:43:07 | serhiy.storchaka | set | keywords:
+ needs review assignee: serhiy.storchaka messages: + msg231655 files: + backslashreplace_decode_2.patch |
| 2014-10-05 04:09:50 | ncoghlan | link | issue22555 dependencies |
| 2014-09-30 06:35:51 | Arfrever | set | nosy:
+ Arfrever |
| 2014-09-23 17:36:22 | serhiy.storchaka | set | dependencies: + Possible integer overflow in error handlers |
| 2014-09-23 12:28:37 | serhiy.storchaka | link | issue22469 superseder |
| 2014-09-23 12:27:37 | serhiy.storchaka | set | files:
+ backslashreplace_decode.patch components: + Extension Modules keywords: + patch nosy: + serhiy.storchaka messages: + msg227352 stage: needs patch -> patch review |
| 2014-08-27 11:00:40 | ncoghlan | link | issue18814 dependencies |
| 2014-08-27 10:53:51 | vstinner | set | nosy:
+ vstinner |
| 2014-08-27 10:50:20 | ncoghlan | create | |