Call not visited in ast.NodeTransformer

[davidcastells]

BPO	46719
Nosy	@isidentical, @davidcastells

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = None
created_at = <Date 2022-02-11.13:29:14.687>
labels = ['type-bug', '3.8', '3.9', '3.10', '3.7', 'library']
title = 'Call not visited in ast.NodeTransformer'
updated_at = <Date 2022-02-11.13:40:39.177>
user = 'https://github.com/davidcastells'

bugs.python.org fields:

activity = <Date 2022-02-11.13:40:39.177>
actor = 'BTaskaya'
assignee = 'none'
closed = False
closed_date = None
closer = None
components = ['Library (Lib)']
creation = <Date 2022-02-11.13:29:14.687>
creator = 'davidcastells'
dependencies = []
files = []
hgrepos = []
issue_num = 46719
keywords = []
message_count = 2.0
messages = ['413069', '413070']
nosy_count = 2.0
nosy_names = ['BTaskaya', 'davidcastells']
pr_nums = []
priority = 'normal'
resolution = None
stage = None
status = 'open'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue46719'
versions = ['Python 3.7', 'Python 3.8', 'Python 3.9', 'Python 3.10']

[davidcastells]

If I create a class derived from ast.NodeTransformer and implement the visit_Call.
When run on the below code, the visit_Call function is only called once (for the print function, and not for ord). It looks like calls in function arguments are ignored.

def main():
    print(ord('A'))

On the other hand, on the following code it correctly visits both functions (print and ord).

def main():
    c = org('A')
    print(c)

[isidentical]

It looks like calls in function arguments are ignored.

Please share a small, self-contained reproducer.

From what I can assume by this message, the problem is that you are not calling self.generic_visit(node) on the first call you are handling. If you don't call visit/generic_visit on the rood node that you are handling (print(ord('A')), then it will never visit ord('A') in a standalone fashion.

[hauntsaninja]

Feel free to re-open if a) your issue is something other than what isidentical mentioned, b) you have a reproducer

[AlexWaygood]

Feel free to re-open if a) your issue is something other than what isidentical mentioned, b) you have a reproducer

Cc. @davidcastells (I believe "mannequin" users aren't autosubscribed to issues they opened on BPO, even if they're listed on the nosy list)