Skip to content

Revert "Add line # of previous definition" #3422

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

Merged
merged 6 commits into from
May 23, 2017
Merged

Conversation

gvanrossum
Copy link
Member

Reverts #3396

This caused a new crash, the rollback fixes #3415.

@JukkaL
Copy link
Collaborator

JukkaL commented May 23, 2017

Let's merge this instead #3420; @pkch will resubmit an updated version of the original PR with the crash fixed.

@gvanrossum
Copy link
Member Author

OK, did you double-check that both repros from #3415 no longer crash?

@pkch
Copy link
Contributor

pkch commented May 23, 2017

Yes, double-checked, crash no longer occurs.

@JukkaL JukkaL merged commit 71efcdb into master May 23, 2017
@gvanrossum gvanrossum deleted the revert-3396-dupdef branch May 23, 2017 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Internal error
3 participants