I noticed a "FIXME" comment in there. Didn't we do a pass a while back and convert all of them into JBS entries?
The new code simply repeats what was done for the existing Exception which is what has the "FIXME" comment so I would use this opportunity to upgrade that FIXME into an actual bug entry.
Also, perhaps we could use the multi-catch syntax here instead of creating a new catch? ...jim On 8/3/17 2:01 AM, Prasanta Sadhukhan wrote:
Hi All, Please review a fix http://cr.openjdk.java.net/~psadhukhan/fx/8133329/webrev.00/ for issue https://bugs.openjdk.java.net/browse/JDK-8133329 More information in JBS. Regards Prasanta