Hi, Nasser, Joe,
I agree with Joe as to the assessment on DecimalFormat code. The
existing code looks correct. Unless there is any obvious need to replace
the piece with bit operation, I'd just leave it as is. The jira issue
can be transferred to HotSpot, though.
Naoto
On 9/26/18 11:14 AM,
Hi Joe, Naoto,
Thank you for your review comments. I agree with you that as per the VM
specification, divide-by-zero in DecimalFormat is allowed and hence the
current fix is more of a good to have rather than a defect fix. As Joe
suggested, I will see whether the new fix give better performance