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: mailbox: use ASCII to read/write .mh_sequences files
Type: Stage: resolved
Components: Library (Lib) Versions: Python 3.3
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: jerub, python-dev, r.david.murray, vstinner
Priority: normal Keywords: patch

Created on 2011-06-30 16:35 by vstinner, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
mailbox.patch vstinner, 2011-06-30 16:35 review
Messages (3)
msg139502 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2011-06-30 16:35
The MH class uses the locale encoding to read and write .mh_sequences files. According to R. David Murray, ASCII can be safetly used instead. Extract of our discussion on IRC:

bitdancer> haypo: that's one of the cases where using locale
           is probably correct, even though I'd rather it
           used utf-8.
bitdancer> haypo: for nmh it is ASCII, for whatever that is worth.
bitdancer> I base this on the fact that the pick command says
           that a utf-8 character is invalid.
bitdancer> for a sequence name.
bitdancer> Which is the only thing other than numbers that goes
           in a sequences files.
Jerub> I concurr (was reading code)
msg145750 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2011-10-17 18:44
New changeset 5ea81e4c58a7 by Victor Stinner in branch 'default':
Close #12454: The mailbox module is now using ASCII, instead of the locale
http://hg.python.org/cpython/rev/5ea81e4c58a7
msg145751 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2011-10-17 18:44
I kept Python 2.7 and 3.2 unchanged because I'm not completly sure that the fix is correct :-)
History
Date User Action Args
2022-04-11 14:57:19adminsetgithub: 56663
2011-10-17 18:44:43vstinnersetmessages: + msg145751
2011-10-17 18:44:00python-devsetstatus: open -> closed

nosy: + python-dev
messages: + msg145750

resolution: fixed
stage: resolved
2011-06-30 16:36:08vstinnersetnosy: + jerub
2011-06-30 16:35:52vstinnercreate