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

#26506: hex() documentation: mention "%x" % int

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 years, 8 months ago by vstinner
Modified:
3 years ago
Reviewers:
julien+python
CC:
haypo, eric.smith, ezio.melotti, docs_python.org, storchaka, wolma, Manvi B, Mariatta, sharanyalburgi_gmail.com
Visibility:
Public.

Patch Set 1 #

Patch Set 2 #

Patch Set 3 #

Total comments: 8

Patch Set 4 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats Patch
Doc/library/functions.rst View 1 2 3 3 chunks +51 lines, -10 lines 1 comment Download

Messages

Total messages: 2
victor.stinner_gmail.com
http://bugs.python.org/review/26506/diff/16795/Doc/library/functions.rst File Doc/library/functions.rst (right): http://bugs.python.org/review/26506/diff/16795/Doc/library/functions.rst#newcode87 Doc/library/functions.rst:87: '0b11' I suggest to use less boring numbers in ...
3 years, 8 months ago #1
mdk
3 years ago #2
I'm not fan of "If prefix "0b" is desired or not", I'm not a native english but
I think it should be changed.

http://bugs.python.org/review/26506/diff/16805/Doc/library/functions.rst
File Doc/library/functions.rst (right):

http://bugs.python.org/review/26506/diff/16805/Doc/library/functions.rst#newc...
Doc/library/functions.rst:92: If prefix "0b" is desired or not, you can use
either of the following ways. See
I'm not a big fan of "is desired or not", why not avoiding it with something
like:

"When more flexibility is needed, :func:`format` can also format hexadecimal
strings:"

(If so, please change it on the three different places for consistency).
Sign in to reply to this message.

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