Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(264231)

#12832: The documentation for the print function should explain/point to how to control the sys.stdout encoding

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years ago by rdmurray
Modified:
7 years, 9 months ago
Reviewers:
merwok, ezio.melotti
CC:
eric.araujo, r.david.murray, docs_python.org, mach1723_gmail.com
Visibility:
Public.

Patch Set 1 #

Total comments: 7

Patch Set 2 #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
Doc/library/functions.rst View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 2
eric.araujo
Thanks for the patch. The gist is good, I only have style remarks. http://bugs.python.org/review/12832/diff/3254/10126 File ...
8 years ago #1
ezio.melotti
8 years ago #2
http://bugs.python.org/review/12832/diff/3254/10126
File Doc/library/functions.rst (right):

http://bugs.python.org/review/12832/diff/3254/10126#newcode943
Doc/library/functions.rst:943: The default encoding used for the various system
streams(:data:`sys.stdout`,
Missing space after 'streams'

http://bugs.python.org/review/12832/diff/3254/10126#newcode948
Doc/library/functions.rst:948: and if that fails the encoding defaults to ascii.
This sentence is a bit too long.  Maybe you can just say that the encoding is
searched in these 3 places in order before defaulting to ASCII.
Sign in to reply to this message.

RSS Feeds Recent Issues | This issue
This is Rietveld 894c83f36cb7+