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 crash caused by deferred lambdas by not deferring lambdas #4232

Merged
merged 2 commits into from Nov 9, 2017

Conversation

Projects
None yet
2 participants
@JukkaL
Collaborator

JukkaL commented Nov 9, 2017

The inferred type of lambda may affect the surrounding scope, and
lambdas inherit the binder from the surrounding scope. These things
make it a bad idea to defer lambdas.

Fixes #3672.

Fix crash caused by deferred lambdas by not deferring lambdas
The inferred type of lambda may affect the surrounding scope, and
lambdas inherit the binder from the surrounding scope. These things
make it a bad idea to defer lambdas.

Fixes #3672.
@gvanrossum

LGTM assuming testpr is OK with it.

Show outdated Hide outdated mypy/checkexpr.py Outdated

@gvanrossum gvanrossum merged commit 45ca578 into master Nov 9, 2017

1 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@gvanrossum gvanrossum deleted the lambda-crash branch Nov 9, 2017

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