[issue39639] Remove Suite node from AST

Change by Pablo Galindo Salgado : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue39639] Remove Suite node from AST

Pablo Galindo Salgado added the comment: New changeset d82e469048e0e034d8c0020cd33b733be1adf68b by Batuhan Taşkaya in branch 'master': bpo-39639: Remove the AST "Suite" node and associated code (GH-18513) https://github.com/python/cpython/commit/d82e469048e0e034d8c0020cd33b733be1adf68b -

[issue39639] Remove Suite node from AST

STINNER Victor added the comment: Jeff Allen: thanks for the useful feedback on how it's used in Jython! -- nosy: +vstinner ___ Python tracker ___

[issue39639] Remove Suite node from AST

Jeff Allen added the comment: Jython uses the reference grammar and ASDL as a way to ensure it is Python we approximate, not some subtly different language. The presence of Suite here gives rise to a class (https://github.com/jythontools/jython/blob/v2.7.2b3/src/org/python/antlr/ast/Suite.ja

[issue39639] Remove Suite node from AST

hai shi added the comment: I am not sure there have any relation with jython's python.asdl? https://github.com/jythontools/jython/blob/master/ast/Python.asdl#L10 -- nosy: +shihai1991 ___ Python tracker

[issue39639] Remove Suite node from AST

Change by Batuhan Taskaya : -- keywords: +patch pull_requests: +17890 stage: -> patch review pull_request: https://github.com/python/cpython/pull/18513 ___ Python tracker ___

[issue39639] Remove Suite node from AST

Change by Batuhan Taskaya : -- priority: normal -> low title: Remote Suite node from AST -> Remove Suite node from AST ___ Python tracker ___ __