This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: devguide: compiler - poor wording
Type: enhancement Stage: resolved
Components: Devguide Versions:
process
Status: closed Resolution: duplicate
Dependencies: Superseder: devguide: compiler - wording
View: 19316
Assigned To: ezio.melotti Nosy List: ezio.melotti, numerodix
Priority: normal Keywords: patch

Created on 2013-10-20 12:12 by numerodix, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
issue19315.diff ezio.melotti, 2013-10-21 00:36
Messages (3)
msg200570 - (view) Author: Martin Matusiak (numerodix) * Date: 2013-10-20 12:12
Location:
http://docs.python.org/devguide/compiler.html#parse-trees

- To tie all of this example, consider the rule for ‘while’:

Probably meant to be: To tie all of this together with an example, ...

- The node representing this will have TYPE(node) == while_stmt and the number of children can be 4 or 7 depending on if there is an ‘else’ statement.

s/if/whether/g is slightly better English imo

- To access what should be the first ‘:’ and require it be an actual ‘:’ token, (REQ(CHILD(node, 2), COLON)`.

Understandable with some effort on the reader's part, but it's not very well worded.

Notice also the closing backtick: markup typo.
msg200674 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2013-10-21 00:36
Here is a patch.
msg200677 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2013-10-21 00:45
This can be fixed as part of #19316.
History
Date User Action Args
2022-04-11 14:57:52adminsetgithub: 63514
2013-10-21 00:45:11ezio.melottisetstatus: open -> closed
resolution: duplicate
messages: + msg200677

superseder: devguide: compiler - wording
stage: patch review -> resolved
2013-10-21 00:36:25ezio.melottisetfiles: + issue19315.diff
messages: + msg200674

assignee: ezio.melotti
keywords: + patch
type: enhancement
stage: patch review
2013-10-20 12:12:53numerodixcreate