[Libreoffice] [PATCH] Basic Currency Issues #i31001# to #i107277#

Noel Power nopower at novell.com
Tue Nov 30 03:22:15 PST 2010


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 regenerated the patch to do
that. 

regards,
Noel

p.s. are you aware of more iz issues related to this other than the 2
mentioned in the subject? be good to have as much test data as possible




More information about the LibreOffice mailing list