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

bpo-39080: Starred Expression's column offset fix when inside a CALL #17645

Open
wants to merge 4 commits into
base: master
from

Conversation

@lysnikolaou
Copy link
Contributor

lysnikolaou commented Dec 17, 2019

….OrxEVS.rst

Co-Authored-By: Pablo Galindo <Pablogsal@gmail.com>
@lysnikolaou lysnikolaou requested a review from pablogsal Dec 17, 2019
@pablogsal

This comment has been minimized.

Copy link
Member

pablogsal commented Dec 17, 2019

Actually, this seems that does not only affect starred expressions. For example:

>>> val = "f(a,b,c)"
>>> val[ast.parse(val).body[0].value.args[2].end_col_offset]
')'
>>> val[ast.parse(val).body[0].value.args[0].end_col_offset]
','
>>> val[ast.parse(val).body[0].value.args[1].end_col_offset]
','
>>> val[ast.parse(val).body[0].value.args[2].end_col_offset]
')'

that seems odd.

@gvanrossum

This comment has been minimized.

Copy link
Member

gvanrossum commented Dec 17, 2019

Possibly @ilevkivskyi would also be interested in hearing about this?

@pablogsal pablogsal requested a review from ilevkivskyi Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.