Skip to content

bpo-36218: Fix handling of heterogeneous values in list.sort #12209

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.

Sign up for GitHub

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

Merged
rhettinger merged 9 commits into from
Mar 25, 2019
Merged

bpo-36218: Fix handling of heterogeneous values in list.sort #12209

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
e75d9fb
Fix sorting of heterogeneous values in list.sort
Mar 7, 2019
8498a21
Add blurb
Mar 7, 2019
60aca1e
Remove early exit when keys are tuples
Mar 12, 2019
eaa7c50
Fix bug found by test_json
Mar 12, 2019
3b02a05
Remove empty line
Mar 12, 2019
47a0688
Simplify logic in pre-sort check
Mar 21, 2019
467cd16
Add test
Mar 21, 2019
8b2224f
Fix coding style
serhiy-storchaka Mar 21, 2019
6286a4b
Fix indentation
Mar 22, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add test
  • Loading branch information
Rémi Lapeyre committed Mar 21, 2019
commit 467cd16465ff001a42d97d037eacf4aa6f35013a
1 change: 1 addition & 0 deletions Lib/test/test_sort.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -377,6 +377,7 @@ def test_unsafe_tuple_compare(self):
def test_not_all_tuples(self):
self.assertRaises(TypeError, [(1.0, 1.0), (False, "A"), 6].sort)
self.assertRaises(TypeError, [('a', 1), (1, 'a')].sort)
Comment thread
remilapeyre marked this conversation as resolved.
self.assertRaises(TypeError, [(1, 'a'), ('a', 1)].sort)
#==============================================================================

if __name__ == "__main__":
Expand Down