Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork3.1k
Fix and optimise overload compatibility checking#14018
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Discovered as part ofpython#14017
This comment has been minimized.
This comment has been minimized.
| previous_match_left_index=left_index | ||
| found_match=True | ||
| matched_overloads.add(left_index) | ||
| break |
hauntsaninjaNov 6, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
adding the break here is the main functional change, everything in the second commit preserves behaviour
According tomypy_primer, this change has no effect on the checked open source code. 🤖🎉 |
Uh oh!
There was an error while loading.Please reload this page.
Discovered as part of#14017