New submission from STINNER Victor victor.stin...@haypocalc.com:
Attached patch document PyUnicode_DecodeFSDefault() and
PyUnicode_DecodeFSDefaultAndSize() in Doc/c-api/, and fix comment in
Include/unicodeobject.c (remove reference to bytearray, and specify that
surrogateescape is used).
The
Éric Araujo mer...@netwok.org added the comment:
The bulk of the patch seems good to me, with some minor remarks:
- You haven’t used title case consistently in all section titles.
- I’d just say “wchar_t Support” in a title, deleting “for platforms
which support it”.
- You could mark up “bytes”
Antoine Pitrou pit...@free.fr added the comment:
Is it better to say “for encoding x, use y” or “to encode x,
use y”?
The latter, IMO. Encoding is also a noun.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
Éric Araujo mer...@netwok.org added the comment:
In this case, it was a verb (a gerund form).
--
title: Document PyUnicode_DecodeFSDefault() and
PyUnicode_DecodeFSDefaultAndSize() - Document PyUnicode_DecodeFSDefault()
and PyUnicode_DecodeFSDefaultAndSize()
STINNER Victor victor.stin...@haypocalc.com added the comment:
Thanks for your remarks: updated patch.
--
Added file: http://bugs.python.org/file17339/unicode_doc-2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8711
STINNER Victor victor.stin...@haypocalc.com added the comment:
Commited: r81168 (py3k), r81169 (3.1).
I also added the paragraph titles to Python2: r81166 (trunk) and r81167 (2.6).
--
resolution: - fixed
status: open - closed
___
Python tracker
