mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-09-26 18:29:57 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

Classes that override __eq__ also need to define __hash__.

Browse source
This commit is contained in:
Raymond Hettinger 2010-04-05 18:53:43 +00:00
parent 5e0c2748fb
commit e1d665a90e
2 changed files with 9 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Lib/functools.py
View file

@ -93,4 +93,6 @@ def cmp_to_key(mycmp):
return mycmp(self.obj, other.obj) >= 0 return mycmp(self.obj, other.obj) >= 0
def __ne__(self, other): def __ne__(self, other):
return mycmp(self.obj, other.obj) != 0 return mycmp(self.obj, other.obj) != 0
def __hash__(self):
raise TypeError('hash not implemented')
return K return K

7
Lib/test/test_functools.py
View file

@ -345,6 +345,13 @@ class TestCmpToKey(unittest.TestCase):
self.assertEqual(sorted(range(5), key=functools.cmp_to_key(mycmp)), self.assertEqual(sorted(range(5), key=functools.cmp_to_key(mycmp)),
[4, 3, 2, 1, 0]) [4, 3, 2, 1, 0])
def test_hash(self):
def mycmp(x, y):
return y - x
key = functools.cmp_to_key(mycmp)
k = key(10)
self.assertRaises(TypeError, hash(k))
class TestTotalOrdering(unittest.TestCase): class TestTotalOrdering(unittest.TestCase):
def test_total_ordering_lt(self): def test_total_ordering_lt(self):