➜
This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on2014-08-27 10:50 byncoghlan, last changed2022-04-11 14:58 byadmin. This issue is nowclosed.
| 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 onissue 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 withissue22470 andissue19676). | |||
| 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 changesetdd8a03e98158 by Serhiy Storchaka in branch 'default':Issue#22286: The "backslashreplace" error handlers now works withhttps://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. Seeissue23321 for this. | |||
| msg234784 -(view) | Author: Roundup Robot (python-dev) | Date: 2015-01-26 22:27 | |
New changeset1e8937861ee3 by Victor Stinner in branch 'default':Issue#22286,#23321: Fix failing test on Windows code page 932https://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 | |