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

addEnd returns incorrect end_col #1

Closed
Varal7 opened this issue Mar 19, 2019 · 4 comments
Closed

addEnd returns incorrect end_col #1

Varal7 opened this issue Mar 19, 2019 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@Varal7
Copy link

Varal7 commented Mar 19, 2019

The current implementation of addEnd returns an incorrect end_col. It is based on the first_non_child of the node. I can't say I understand what is going on but the result is incorrect. For instance, using webdiff:

Screen Shot 2019-03-19 at 3 49 50 PM

@jrfaller
Copy link
Member

Hi! I tried to improve pythonparser some time ago. Can you try with the new version? Cheers!

@Varal7
Copy link
Author

Varal7 commented Nov 19, 2019

Hi, actually I’ve been using the PR I made in #2, which fixes this.

@jrfaller
Copy link
Member

jrfaller commented Nov 19, 2019

Wow awesome! I didn't know there was someone working on this since I do not go very often on pythonparser's repo. I committed the code but I am not the original author. Last commit I made was to make pythonparser compatible with the official GumTree XML exchange format to simplify its integration in GumTree. Along the way I simplified the code a bit and made some fixes. But surely, your version is better than mine, so it would make sense to replace my version by yours. Do you think you could adapt the output to produce a standard GumTree XML? I could help you to show you how it works (for instance it is explained here : GumTreeDiff/gumtree#164).

@jrfaller jrfaller self-assigned this Apr 27, 2021
@jrfaller jrfaller added the bug Something isn't working label Apr 27, 2021
@jrfaller
Copy link
Member

Closing this since we now use Parso for parsing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants