Re: [9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying
I'm not familiar with this area. The changes looks reasonable enough to me. -Brent On 06/27/2016 01:19 PM, Kumar Srinivasan wrote: 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.
Re: [9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying
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.
Re: [9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying
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.
Re: [9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying
> 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-