(original) (raw)

changeset: 73091:221638ba5d2a user: Mark Dickinson mdickinson@enthought.com date: Mon Oct 24 10:31:52 2011 +0100 files: Lib/decimal.py Lib/test/test_decimal.py Misc/NEWS description: Issue #13248, issue #8540: Remove deprecated Context._clamp attribute from Decimal module. diff -r 4eb65a7f2fbe -r 221638ba5d2a Lib/decimal.py --- a/Lib/decimal.py Mon Oct 24 00:25:41 2011 +0200 +++ b/Lib/decimal.py Mon Oct 24 10:31:52 2011 +0100 @@ -3903,28 +3903,6 @@ return nc __copy__ = copy - # _clamp is provided for backwards compatibility with third-party - # code. May be removed in Python >= 3.3. - def _get_clamp(self): - "_clamp mirrors the clamp attribute. Its use is deprecated." - import warnings - warnings.warn('Use of the _clamp attribute is deprecated. ' - 'Please use clamp instead.', - DeprecationWarning) - return self.clamp - - def _set_clamp(self, clamp): - "_clamp mirrors the clamp attribute. Its use is deprecated." - import warnings - warnings.warn('Use of the _clamp attribute is deprecated. ' - 'Please use clamp instead.', - DeprecationWarning) - self.clamp = clamp - - # don't bother with _del_clamp; no sane 3rd party code should - # be deleting the _clamp attribute - _clamp = property(_get_clamp, _set_clamp) - def _raise_error(self, condition, explanation = None, *args): """Handles an error diff -r 4eb65a7f2fbe -r 221638ba5d2a Lib/test/test_decimal.py --- a/Lib/test/test_decimal.py Mon Oct 24 00:25:41 2011 +0200 +++ b/Lib/test/test_decimal.py Mon Oct 24 10:31:52 2011 +0100 @@ -1834,18 +1834,9 @@ # only, the attribute should be gettable/settable via both # `clamp` and `_clamp`; in Python 3.3, `_clamp` should be # removed. - c = Context(clamp = 0) - self.assertEqual(c.clamp, 0) - - with check_warnings(("", DeprecationWarning)): - c._clamp = 1 - self.assertEqual(c.clamp, 1) - with check_warnings(("", DeprecationWarning)): - self.assertEqual(c._clamp, 1) - c.clamp = 0 - self.assertEqual(c.clamp, 0) - with check_warnings(("", DeprecationWarning)): - self.assertEqual(c._clamp, 0) + c = Context() + with self.assertRaises(AttributeError): + clamp_value = c._clamp def test_abs(self): c = Context() diff -r 4eb65a7f2fbe -r 221638ba5d2a Misc/NEWS --- a/Misc/NEWS Mon Oct 24 00:25:41 2011 +0200 +++ b/Misc/NEWS Mon Oct 24 10:31:52 2011 +0100 @@ -338,6 +338,8 @@ Library ------- +- Issue #8540: Remove deprecated Context._clamp attribute in Decimal module. + - Issue #13235: Added PendingDeprecationWarning to warn() method and function. - Issue #9168: now smtpd is able to bind privileged port. /mdickinson@enthought.com