Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ensure the position information in AST nodes created by the parser is always consistent #93351

Closed
pablogsal opened this issue May 30, 2022 · 0 comments · Fixed by #93352
Closed

Ensure the position information in AST nodes created by the parser is always consistent #93351

pablogsal opened this issue May 30, 2022 · 0 comments · Fixed by #93352
Labels
3.10 3.11 3.12 interpreter-core

Comments

@pablogsal
Copy link
Member

@pablogsal pablogsal commented May 30, 2022

If we create an AST node with invalid position information, the error is reported in the compiler when it validates the line numbers and end line numbers, which is too far away and debugging this can be quite challenging. To improve the debugging experience, ensure that the parser cannot create an AST node with invalid position information.
@pablogsal pablogsal mentioned this issue May 30, 2022
Merged
pablogsal added a commit to pablogsal/cpython that referenced this issue May 30, 2022
@pablogsal
pythongh-93351: Ensure the position information in AST nodes created …
6427d3f 
…by the parser is always consistent
@AA-Turner AA-Turner added interpreter-core 3.11 3.10 3.12 labels May 30, 2022
pablogsal added a commit to pablogsal/cpython that referenced this issue May 30, 2022
@pablogsal
pythongh-93351: Ensure the position information in AST nodes created …
7e04d15 
…by the parser is always consistent
pablogsal added a commit to pablogsal/cpython that referenced this issue May 30, 2022
@pablogsal
pythongh-93351: Validate the position information when compiling AST …
842049d 
…nodes
pablogsal added a commit to pablogsal/cpython that referenced this issue May 30, 2022
@pablogsal
pythongh-93351: Validate the position information when compiling AST …
dde7b64 
…nodes
pablogsal added a commit to pablogsal/cpython that referenced this issue May 30, 2022
@pablogsal
fixup! pythongh-93351: Validate the position information when compili…
20731eb 
…ng AST nodes
pablogsal added a commit to pablogsal/cpython that referenced this issue May 30, 2022
@pablogsal
fixup! pythongh-93351: Validate the position information when compili…
8d865bf 
…ng AST nodes
pablogsal added a commit that referenced this issue May 30, 2022
@pablogsal
gh-93351: Ensure the position information in AST nodes created by the…
5893b5d 
… parser is always consistent (GH-93352)
miss-islington pushed a commit to miss-islington/cpython that referenced this issue May 30, 2022
@pablogsal @miss-islington
pythongh-93351: Ensure the position information in AST nodes created …
9311b89 
…by the parser is always consistent (pythonGH-93352)

(cherry picked from commit 5893b5d)

Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
miss-islington added a commit that referenced this issue May 30, 2022
@miss-islington @pablogsal
gh-93351: Ensure the position information in AST nodes created by the…
7f6e6ab 
… parser is always consistent (GH-93352)

(cherry picked from commit 5893b5d)

Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3.10 3.11 3.12 interpreter-core
Projects
None yet
Milestone
No milestone
2 participants
@AA-Turner @pablogsal