[issue14478] Decimal hashing very slow, could be cached

Jim Jewett report at bugs.python.org
Tue Apr 10 16:16:45 CEST 2012


Jim Jewett <jimjjewett at gmail.com> added the comment:

Stefan Krah has a good point.  

Since the only cost is an extra slot, and this is for users who have already chosen to use Decimal instead of a more efficient (but possibly less accurate) representation, even without the native speedups to Decimal ... I guess I'm now +1.  

It is clearly an implementation detail, so I don't think the docs need to be changed.  I'm not sure the tests do, nor am I sure *how* to test for it in a black-box fashion.

I have reviewed the patch and have no objections.

So unless a new objection comes up in the next few days, I would say this is ready to be checked in.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue14478>
_______________________________________


More information about the Python-bugs-list mailing list