[Libreoffice] [PATCH] Basic Currency Issues #i31001# to #i107277#
Noel Power
nopower at novell.com
Tue Nov 30 03:26:25 PST 2010
sending again ( whilst attempting to preserve the threading this time )
On Tue, 2010-11-30 at 01:35 -0800, John LeMoyne Castle wrote:
> OOo Basic has had issues with its fixed point Currency type for a long
time.
> I think the Basic Currency type deficiencies create a bar to adoption
in the
> minds of business users.
> Nothing says 'run away' like mangling the money numbers.
Agreed !! and thanks for looking at this
>
> Currency is presently implemented with a 64bit struct of 2 Int32s and
the
> math is done by the tools/bigint class.
> The attached patches replace the code to implement the Currency type
with an
> ISO C99 compiler supported 64 bit integer.
> In fact, the 64 bit int was already a member of the Sbx core data
union.
> Main patch makes 1/2 kLOC reduction in sbx.
yeah, so you will have to give me some time to look at this ( not really
familar with the Currently type ) and I need to somehow filter out the
whitespace changes too ( probably I can regenerate the patch to do
that. )
regards,
Noel
More information about the LibreOffice
mailing list