Re: [9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying

2016-06-27 Thread Brent Christian
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 pleas

Re: [9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying

2016-06-27 Thread Kumar Srinivasan
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/JD

Re: [9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying

2016-06-23 Thread David DeHaven
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.

Re: [9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying

2016-06-22 Thread Sergey Bylokhov
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 stuffe

Re: [9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying

2016-06-21 Thread David DeHaven
> 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 NSBlo

[9] RFR: 8022291: Mac OS: Unexpected JavaLaunchHelper message displaying

2016-06-21 Thread David DeHaven
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 c