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-38903: Remove #if block that is not used on parsetok.c #17365

Merged
merged 1 commit into from
Jan 13, 2020

Conversation

eamanu
Copy link
Contributor

@eamanu eamanu commented Nov 24, 2019

On parsetok.c we can see a #if that has not effect on compilation

https://bugs.python.org/issue38903

On parsetok.c we can see a #if that has not effect on compilation
Copy link
Member

@shihai1991 shihai1991 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No code call this function, so LGTM.

@eamanu
Copy link
Contributor Author

eamanu commented Nov 25, 2019

I ping to @brettcannon because on commit e3944a5 added it, to understand the context and to know if is safety remove it.

@brettcannon
Copy link
Member

@eamanu best I can think of is it was left there as an example. Otherwise that was over a decade ago so 🤷‍♂️

@pablogsal pablogsal merged commit d23f782 into python:master Jan 13, 2020
sthagen added a commit to sthagen/python-cpython that referenced this pull request Jan 13, 2020
Remove unused functions in Parser/parsetok.c (pythonGH-17365)
@eamanu eamanu deleted the fix-38903 branch January 13, 2020 14:08
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants