Hi, Anton.
The fix looks good. But can you rename the test to some useful name like other tests in java/awt/image?

On 5/29/14 9:11 PM, anton nashatyrev wrote:
Hello,

    any volunteers to review please? The fix is pretty simple.

http://cr.openjdk.java.net/%7Eanashaty/8032788/9/webrev.00/
https://bugs.openjdk.java.net/browse/JDK-8032788

Thanks!
Anton.

On 22.05.2014 22:29, anton nashatyrev wrote:
Hello,
    could you please review the following fix:

fix: http://cr.openjdk.java.net/~anashaty/8032788/9/webrev.00/ <http://cr.openjdk.java.net/%7Eanashaty/8032788/9/webrev.00/>
bug: https://bugs.openjdk.java.net/browse/JDK-8032788

Problem: MediaTracker.waitForID() hangs if the Image was created with null argument. The expected behavior is the same as if the non-existent file name is passed, i.e. ERRORED loading status

    Fix: avoid throwing unexpected NPE from FileImageSource.getDecoder()

Thanks!
Anton.



--
Best regards, Sergey.

Reply via email to