Formatting support for decimal library numbers

Trac Comment by notes—1970-01-01T01:38:48.000Z

dup of 813, 2282

Trac Comment by auditor—1970-01-01T01:38:49.000Z

• Mon Oct 2 18:39:39 2006 grhoten changed notes2: assign: "" to "eric", priority: "" to "medium", weeks: "" to "2",

• Thu Nov 2 17:39:13 2006 emmons changed notes

• Thu Nov 2 17:39:13 2006 emmons moved from formatting to returned

• Thu Nov 2 17:39:41 2006 grhoten changed notes2: xref: "" to "813, 2282",

• Thu Nov 2 17:39:41 2006 grhoten moved from returned to formatting

Trac Comment 4 by —2008-03-13T17:24:41.000Z

Adding this - formatting numbers from the big decimal storage format, or parsing strings into the big decimal form - would treat the decimal number storage form as a digit-list-like thing. It enables applications that are independently using the decimal library for decimal computations to use ICU for formatting/parsing. It does not enable decimal computations using ICU APIs.

Whether or not ICU makes some internal use of any decimal library code, it probably should not be in the business of exporting it. Any non-ICU code needing the decimal library should depend on it directly.

Trac Comment 6 by —2010-02-10T17:15:08.000Z

Suggestions:

• please consider using version 368 (just released)

• instead of replacing uprv_ everywhere, add a new file, decIcu.h that contains '#define decNumberSomething uprv_decNumberSomething' and include it before using the headers within ICU

• Instead of changes in multiple files, add a new file decIcuImp.h that contains '#define malloc uprv_malloc' and inculde it in all implementation files, and have it include decIcu.

Trac Comment 8 by —2010-02-27T02:27:08.000Z

decimfmt.h has some commented-out class declarations (they were moved into another file). I recommend removing the dead code.

cmemory.h on line 269 has an ifdef block around a one-line comment. Should this be
removed?

(I'm not done reviewing yet, but wanted to log these comments)