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

Reply via email to