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

Fix an aststrip crash bug #5065

Merged
merged 1 commit into from May 16, 2018

Conversation

Projects
None yet
2 participants
@msullivan
Collaborator

msullivan commented May 16, 2018

Found in Zulip. In some cases (involving errors), two import *s could
list overlapping imported_names.

Fix an aststrip crash bug
Found in Zulip. In some cases (involving errors), two import *s could
list overlapping imported_names.

@msullivan msullivan requested review from JukkaL and ilevkivskyi May 16, 2018

@ilevkivskyi

Yes, aststrip is hard, thanks for fixing!

@ilevkivskyi ilevkivskyi merged commit e42d600 into master May 16, 2018

4 checks passed

continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@ilevkivskyi ilevkivskyi deleted the crash-fix branch May 16, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment