This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author Jim.Jewett
Recipients Amaury.Forgeot.d'Arc, Jim.Jewett, amaury.forgeotdarc, brian.curtin, casevh, ced, eric.smith, eric.snow, jjconti, mark.dickinson, pitrou, rhettinger, skrah, vstinner
Date 2012-03-06.18:23:11
SpamBayes Score 4.8476112e-11
Marked as misclassified No
Message-id <1331058192.41.0.698368281361.issue7652@psf.upfronthosting.co.za>
In-reply-to
Content
(1)  I think this module would benefit greatly from a map explaining what each file does, and perhaps from some reorganization. 

As best I can yet tell, there are about ~130 files, over a dozen directories, but the only ones that directly affect the implementation are a subset (~33) of the *.c and *h files in Modules/_decimal/ (and not subdirectories).  

Even files that do affect the implementation, such as mpdecimal.c, also seem to have functions thrown in just for testing small pieces of functionality, such as Newton Division.

There may also be some code that really isn't needed, except possibly for backwards compatibility, and could be #ifdef'ed or at least commented.  For example, the comments above io.c function _mpd_strneq(const char *s, const char *l, const char *u, size_t n) mention working around the Turkish un/dotted-i problem when lowercasing -- but why is a decimal library even worried about casing?

(2)  Is assembly allowed?  If not, please make it clear that vcdiv64.asm is just an optional speedup, and that the code doesn't rely upon it.

(3)  Are there parts of this library that provide functionality NOT in the current decimal library?  If so, this should be at least documented, and perhaps either removed or exposed.
History
Date User Action Args
2012-03-06 18:23:12Jim.Jewettsetrecipients: + Jim.Jewett, rhettinger, amaury.forgeotdarc, mark.dickinson, pitrou, vstinner, casevh, eric.smith, jjconti, ced, brian.curtin, skrah, Amaury.Forgeot.d'Arc, eric.snow
2012-03-06 18:23:12Jim.Jewettsetmessageid: <1331058192.41.0.698368281361.issue7652@psf.upfronthosting.co.za>
2012-03-06 18:23:11Jim.Jewettlinkissue7652 messages
2012-03-06 18:23:11Jim.Jewettcreate