Re: [9] Review Request for 8146162: [TEST_BUG] sun/awt/shell/ShellFolderMemoryLeak.java fails with Jigsaw

2016-01-12 Thread Sergey Bylokhov
+1 On 12/01/16 21:21, Alexander Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 11/01/16 17:56, Semyon Sadetsky wrote: Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8146162 webrev: http://cr.openjdk.java.net/~ssadetsky/8146162/webrev.00/

Re: [9] Review Request for 8146162: [TEST_BUG] sun/awt/shell/ShellFolderMemoryLeak.java fails with Jigsaw

2016-01-12 Thread Alexander Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 11/01/16 17:56, Semyon Sadetsky wrote: Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8146162 webrev: http://cr.openjdk.java.net/~ssadetsky/8146162/webrev.00/ @modules corrected and -XaddExports:java.desktop/s

[9] Review Request for 8146162: [TEST_BUG] sun/awt/shell/ShellFolderMemoryLeak.java fails with Jigsaw

2016-01-11 Thread Semyon Sadetsky
Hello, Please review fix for JDK9: bug: https://bugs.openjdk.java.net/browse/JDK-8146162 webrev: http://cr.openjdk.java.net/~ssadetsky/8146162/webrev.00/ @modules corrected and -XaddExports:java.desktop/sun.awt.shell=ALL-UNNAMED is added to the spawn processes. Should be commited to 9-repo-j