Re: RFR: JDK-8211161: java.lang.ArithmeticException: divide by zero from java.text.DecimalFormat.format()

2018-09-26 Thread naoto . sato
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,

Re: RFR: JDK-8211161: java.lang.ArithmeticException: divide by zero from java.text.DecimalFormat.format()

2018-09-26 Thread Nasser Ebrahim
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