-
-
Notifications
You must be signed in to change notification settings - Fork 31.4k
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-32411: IDLE: Remove line number sort in browser.py #5011
Conversation
ON HOLD, as explained on the issue. |
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.
The requested 'change' is to wait for circumstances to change ;-).
LGTM. |
@terryjreedy: Please replace |
Thanks @csabella for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7. |
Insertion in line order makes sorting keys by line order unneeded. (cherry picked from commit 1a4d9ff) Co-authored-by: Cheryl Sabella <cheryl.sabella@gmail.com>
GH-13732 is a backport of this pull request to the 3.7 branch. |
@terryjreedy, thank you! |
Insertion in line order makes sorting keys by line order unneeded.
Since the input dictionary is created in line number and dictionaries are guaranteed to be in insertion order, the sort is no longer needed.
https://bugs.python.org/issue32411