Urgent [9], 2nd round, RFR (S) : JDK-8039915 NumberFormat.format() does not consider required no. of fraction digits properly
William Price
wprice at pros.com
Tue Oct 7 16:33:26 UTC 2014
> Please review this 2nd version of the fix taking into account your feedback.
I'm not recognized here as a reviewer, but the code looked OK to me and
it passed my patch test suite.
> I have merged HALF_UP and HALF_DOWN cases into a single switch case.
Good to know. My current patch injects a method call to shim code
immediately after the HALF_UP case label. I need to account for this or
else the applied patch could potentially cause HALF_DOWN to behave the
same as HALF_UP! Thankfully, this is easily detectable in the bytecode
(not that the impact to my patch is of much concern to the core devs,
but it dove-tails nicely).
I'm looking forward to seeing this merged and the bug fixed/closed. For my
company, the sooner this makes it into an 8u update, the better. Thanks, Olivier.
--
William Price
More information about the core-libs-dev
mailing list