You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by bu...@apache.org on 2013/12/23 17:56:06 UTC

[Bug 55730] Fix org.apache.poi.ss.usermodel.BuiltinFormats.java for 0x29-0x2c

https://issues.apache.org/bugzilla/show_bug.cgi?id=55730

--- Comment #7 from Eric Peters <er...@peters.org> ---
I finally found a more authoritive reference on this patch:
http://support.microsoft.com/kb/147942 - lists the standard build int format
strings:


0x2b      _(* #,##0.00_);_(* (#,##0.00);_(* "-"??_);_(@_)
0x2c      _($* #,##0.00_);_($* (#,##0.00);_($* "-"??_);_(@_)


1) you can see that the format strings do include a space after the repeating
character asterixes

2) the 0x2b should not be the accounting $, and the 2x2c should be, exactly
like the fix in my patch

-        putFormat(m, 0x2b, "_(\"$\"* #,##0.00_);_(\"$\"* (#,##0.00);_(\"$\"*
\"-\"??_);_(@_)");
-        putFormat(m, 0x2c, "_(* #,##0.00_);_(* (#,##0.00);_(*
\"-\"??_);_(@_)");
+        putFormat(m, 0x2b, "_(* #,##0.00_);_(* (#,##0.00);_(*
\"-\"??_);_(@_)");
+        putFormat(m, 0x2c, "_(\"$\"* #,##0.00_);_(\"$\"* (#,##0.00);_(\"$\"*
\"-\"??_);_(@_)");

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org