The fix looks good.

 

I suggest to replace wildcard import statements with specific ones in both the 
files in webrev.

 

Regards,

Ajit

 

From: Rahul Singh 
Sent: Tuesday, January 03, 2017 6:17 PM
To: awt-dev@openjdk.java.net
Subject: <AWT Dev> [awt-dev] Request for Approval and Review: JDK-8140329 [TEST 
BUG] java/awt/SplashScreen/FullscreenAfterSplash/FullScreenAfterSplash.java

 

Hi All,
 
Please review the fix for the bug: HYPERLINK 
"https://bugs.openjdk.java.net/browse/JDK-7146533"JDK-8140329 [TEST BUG]
The test java/awt/SplashScreen/FullscreenAfterSplash/FullScreenAfterSplash.java 
fails to create test.png in the given path by GenerateTestImage.java utility. I 
used "user.dir" to obtain the path.
 
Bug Link: HYPERLINK 
"https://bugs.openjdk.java.net/browse/JDK-7146533"https://bugs.openjdk.java.net/browse/JDK-8140329
 
The webrev is : HYPERLINK 
"http://cr.openjdk.java.net/%7Evagarwal/8140329/webrev/"http://cr.openjdk.java.net/~vagarwal/8140329/webrev/
 
Regards,
 
Rahul Singh

Reply via email to