On 1/7/2015 9:39 AM, Brian Burkhalter wrote:
On Jan 7, 2015, at 1:23 AM, Paul Sandoz <paul.san...@oracle.com> wrote:
In java.util.Formatter.BigDecimalLayout.layout() trailing zeroes are
erroneously appended when the unscaled value of the BigDecimal is zero and the
scale is negative. Some other minor changes are included to make the logic in
the method clearer.
Look good.
I minor comment, ignore if you wish. If you wanna clean it up further you can
merge into one if/else if/else sequence.
Thanks for the suggestion; I like that better also. An updated version is here:
http://cr.openjdk.java.net/~bpb/8059175/webrev.01/
For the sake of formality, a +1 on this minor tweak from a Reviewer would be
appreciated.
+1
-Joe