Hi, Alexander.
Fix looks fine. It would be good to fix typo: Fallback value is never used *bacause*.

10.01.2013 17:25, Alexander Scherbatiy wrote:

Hello,

Could you review the fix:
  bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7166409
  webrev: http://cr.openjdk.java.net/~alexsch/7166409/webrev.00

The test passes on JDK 6 because it catches and ignores the "NumberFormatException: multiple points". After fixing the NumberFormatException in JDK 7 the test fails with the NPE because "win.highContrast.on" property returns null. The fix treats the null value of the "win.highContrast.on" property as false.


Thanks,
Alexandr.



--
Best regards, Sergey.

Reply via email to