Message157955
| Author |
Jim.Jewett |
| Recipients |
Jim.Jewett, Jimbofbx, Ramchandra Apte, jcea, pitrou, rhettinger, serhiy.storchaka, skrah |
| Date |
2012年04月10日.14:16:44 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1334067405.21.0.830985954446.issue14478@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
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. |
|