◐ Shell
clean mode source ↗

bpo-32411: IDLE: Remove line number sort in browser.py by csabella · Pull Request #5011 · python/cpython

Conversation

@csabella

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

@terryjreedy

ON HOLD, as explained on the issue.

terryjreedy

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 ;-).

@rhettinger

@terryjreedy terryjreedy changed the title bpo-32411: IDLE: Remove sorting by line number in browser.py bpo-32411: IDLE: Remove line number sort in browser.py

Jun 1, 2019

terryjreedy

@bedevere-bot

@terryjreedy: Please replace # with GH- in the commit message next time. Thanks!

@miss-islington

Thanks @csabella for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request

Jun 1, 2019
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>

@bedevere-bot

miss-islington added a commit that referenced this pull request

Jun 1, 2019
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>

@csabella

DinoV pushed a commit to DinoV/cpython that referenced this pull request

Jan 14, 2020
Insertion in line order makes sorting keys by line order unneeded.

Labels