Éric Araujo 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” so that users get a link. - The verb form of “fall back” is in two words, unless I’m lagging on current technical English. - You lack an article when saying “the "surrogateescape" error handler”. - Not sure: Is it better to say “for encoding x, use y” or “to encode x, use y”? Sorry for not giving precise line numbers or a diff of a diff, that wouldn’t be practical. ---------- nosy: +merwok title: Document PyUnicode_DecodeFSDefault() and PyUnicode_DecodeFSDefaultAndSize() -> Document PyUnicode_DecodeFSDefault() and PyUnicode_DecodeFSDefaultAndSize() _______________________________________ Python tracker http://bugs.python.org/issue8711 _______________________________________

STINNER Victor 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 http://bugs.python.org/issue8711 _______________________________________