I am not an expert in this area, if Sergey is ok with it.
I am fine with it.
Brent ?
Kumar
Thanks, Sergey.
Can someone from the core-libs launcher group please approve? (looks at Kumar)
-DrD-
Looks fine.
On 22.06.16 1:58, David DeHaven wrote:
JBS:
https://bugs.openjdk.java.net/browse/JDK-8022291
Webrev:
http://cr.openjdk.java.net/~ddehaven/8022291/jdk.0/index.html
This actually turned out to be pretty easy to fix, I eliminated the
JavaLaunchHelper class and in place of it stuffed the block it replaced into a
NSBlockOperation then changed the performSelectorOnMainThread call to invoke
the NSBlockOperation's start message.
I tested against the SWT snippet (Snippet297) that was attached to the original
Eclipse bug that triggered the original fix. The SWT tests I could dig up all
seemed to work ok.
Original Eclipse bug, used to verify the fix:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=389486
This should be backported to 8u after it bakes in 9 for a bit.
Minor update:
http://cr.openjdk.java.net/~ddehaven/8022291/jdk.1/
I put the NSAutoreleasePool back, so it's directly portable to jdk8u.
-DrD-
--
Best regards, Sergey.
