Formatting support for decimal library numbers

Description

Deleted Component: formatting

RFE: Add number formatter support for decimal numbers in the storage formats
used by Mike Cowlishaw's decimal library.

The decimal library will be distributed with ICU, for use by applications
needing to use decimal arithmetic for their computations. ICU should be able to
parse in and format out numbers for use with this package.

Activity

Show:
TracBot
June 30, 2018, 11:36 PM
Trac Comment by notes—1970-01-01T01:38:48.000Z

dup of 813, 2282

TracBot
June 30, 2018, 11:36 PM
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

TracBot
June 30, 2018, 11:36 PM
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.

TracBot
June 30, 2018, 11:36 PM
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.

TracBot
June 30, 2018, 11:36 PM
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)

Fixed

Assignee

Andy Heninger

Reporter

TracBot

Components

None

Labels

Reviewer

None

Priority

minor

Time Needed

Weeks

Fix versions