Issue #10356: hash(Decimal("sNaN")) now raises ValueError instead of TypeError.

cloud-tester · cloud-tester · commit d8b661dd9086 · 2010-11-18T15:20:34.000Z
diff --git a/Lib/decimal.py b/Lib/decimal.py
@@ -943,7 +943,7 @@ def __hash__(self):
         # in the documentation.  (See library docs, 'Built-in Types').
         if self._is_special:
             if self.is_snan():
-                raise TypeError('Cannot hash a signaling NaN value.')
+                raise ValueError('Cannot hash a signaling NaN value.')
             elif self.is_nan():
                 return _PyHASH_NAN
             else:
diff --git a/Lib/test/test_decimal.py b/Lib/test/test_decimal.py
@@ -1346,7 +1346,7 @@ def hashit(d):
 
         #the same hash that to an int
         self.assertEqual(hashit(Decimal(23)), hashit(23))
-        self.assertRaises(TypeError, hash, Decimal('sNaN'))
+        self.assertRaises(ValueError, hash, Decimal('sNaN'))
         self.assertTrue(hashit(Decimal('Inf')))
         self.assertTrue(hashit(Decimal('-Inf')))
 
diff --git a/Misc/NEWS b/Misc/NEWS
@@ -104,6 +104,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #10356: hash(Decimal("sNaN")) now raises ValueError instead of
+  TypeError.
+
 - Issue #10356: Decimal.__hash__(-1) should return -2.
 
 - Issue #1553375: logging: Added stack_info kwarg to display stack information.
