[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-25 Thread Mariatta Wijaya
Mariatta Wijaya added the comment: The proposed change seems fine. Merged and backported. Thanks everyone! -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-25 Thread Mariatta Wijaya
Mariatta Wijaya added the comment: New changeset 26b940f762f0b306ff894601003f51a8d036e285 by Mariatta (Miss Islington (bot)) in branch '3.6': bpo-31571: Remove duplicated info in Lexical Analysis documentation (GH-3691) (GH-3754) https://github.com/python/cpython/commit/26b940f762f0b306ff89460

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-25 Thread Roundup Robot
Changes by Roundup Robot : -- keywords: +patch pull_requests: +3741 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-25 Thread Mariatta Wijaya
Mariatta Wijaya added the comment: New changeset 4a2d00cb4525fcb3209f04531472ba6a359ed418 by Mariatta (Guilherme Caminha) in branch 'master': bpo-31571: Remove duplicated info in Lexical Analysis documentation (GH-3691) https://github.com/python/cpython/commit/4a2d00cb4525fcb3209f04531472ba6a35

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-25 Thread Mariatta Wijaya
Changes by Mariatta Wijaya : -- stage: -> patch review versions: +Python 3.6 ___ Python tracker ___ ___ Python-bugs-list mailing lis

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-25 Thread Guilherme Praciano Karst Caminha
Guilherme Praciano Karst Caminha added the comment: In that case, I believe the PR I've opened should be fine? -- ___ Python tracker ___

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-25 Thread Guido van Rossum
Guido van Rossum added the comment: That's *really* old. Please clean up as you see fit! On Sep 24, 2017 10:33 PM, "Serhiy Storchaka" wrote: Serhiy Storchaka added the comment: Guido, in 60f2f0cf8e10c94693dfea8937b7feabeffe5744 you added the phrase "Note that numeric literals do not include a

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-24 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Guido, in 60f2f0cf8e10c94693dfea8937b7feabeffe5744 you added the phrase "Note that numeric literals do not include a sign; ..." in the "Numeric literals" section. Did you mean moving it from the "Floating point literals" section or creating a duplicate? ---

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-24 Thread Guido van Rossum
Guido van Rossum added the comment: Is there anything here that requires my attention? -- ___ Python tracker ___ ___ Python-bugs-list

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-24 Thread Guilherme Praciano Karst Caminha
Guilherme Praciano Karst Caminha added the comment: Also, "Numeric literals" is a more general section, so rules that apply for the three types of numeric literals should probably go there to avoid repetition. Maybe the Integer, Floating point and Imaginary literal sections should be one secti

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-24 Thread Guilherme Praciano Karst Caminha
Guilherme Praciano Karst Caminha added the comment: I believe that the sentence regarding the - operator also applies to Integer literals and Imaginary literals. But it only appears on the Floating point literals and Numeric literals sections. -- __

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-24 Thread Mariatta Wijaya
Mariatta Wijaya added the comment: I think we can remove ``` As of Python 3.3 it is possible again to prefix ..." ``` and keep the other one where it says: ``` .. versionadded:: 3.3 Support for the unicode legacy literal ``` About the sentence about numeric literals that appears twice:

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-24 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: The first pair was added in 50364b4a5c8f02ec05d33928e29a8780d9acf968 by Armin Ronacher. The duplicate of the other phrase was introduced in 60f2f0cf8e10c94693dfea8937b7feabeffe5744. -- nosy: +aronacher, gvanrossum, serhiy.storchaka

[issue31571] Redundand information on Doc/reference/lexical_analysis.rst

2017-09-24 Thread Guilherme Praciano Karst Caminha
New submission from Guilherme Praciano Karst Caminha: The file Doc/reference/lexical_analysis.rst has at least two redundant parts. The first one is quoted below: "As of Python 3.3 it is possible again to prefix string literals with a u prefix to simplify maintenance of dual 2.x and 3.x codebas