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.

Thanks,

Brian

Reply via email to