Re: RFR [9] 8154017: Shutdown hooks are racing against shutdown sequence, if System.exit()-calling thread is interrupted

2016-06-23 Thread David Holmes
Looks good. I would probably have had the hook interrupt the main thread while it was running ie sleep(5000); main.interrupt(); sleep(5000); but pre-interrupting itself before calling exit is probably functionally equivalent. David On 24/06/2016 2:32 AM, Chris Hegarty wrote: The shutdown h

Re: RFR 8159751: ObjectStreamClass: hide 'final' flag for anonymous classes

2016-06-23 Thread Roger Riggs
Hi Dan, On 6/23/2016 5:03 PM, Dan Smith wrote: On Jun 23, 2016, at 2:04 PM, Roger Riggs wrote: Hi Dan, On 6/23/2016 2:53 PM, Dan Smith wrote: On Jun 23, 2016, at 11:37 AM, Roger Riggs wrote: Hi Dan, I was concerned about inter-operation between versions because serialized objects are p

Re: RFR 8159751: ObjectStreamClass: hide 'final' flag for anonymous classes

2016-06-23 Thread Dan Smith
> On Jun 23, 2016, at 2:04 PM, Roger Riggs wrote: > > Hi Dan, > > On 6/23/2016 2:53 PM, Dan Smith wrote: >>> On Jun 23, 2016, at 11:37 AM, Roger Riggs >>> wrote: >>> >>> Hi Dan, >>> >>> I was concerned about inter-operation between versions because serialized >>> objects >>> are passed betw

Re: RFR 9 7131356 : (props) "No Java runtime present, requesting install" when creating VM from JNI [macosx]

2016-06-23 Thread Naoto Sato
+1 > On 23/6/16 上午10:34, Naoto Sato wrote: I see that you tested thoroughly :-) Naoto On 6/23/16 1:13 PM, Brent Christian wrote: On 23/6/16 上午10:34, Naoto Sato wrote: I reviewed webrev.04 thoroughly this time and comments below: - Although the OSX API returns the cases as in the spec, BCP 4

Re: RFR 9 7131356 : (props) "No Java runtime present, requesting install" when creating VM from JNI [macosx]

2016-06-23 Thread Brent Christian
On 23/6/16 上午10:34, Naoto Sato wrote: I reviewed webrev.04 thoroughly this time and comments below: - Although the OSX API returns the cases as in the spec, BCP 47 language tags are case insensitive, so it would be better to check [a-z] in line 94/95 as well. Done (and I've simplified the ass

Re: RFR 8159751: ObjectStreamClass: hide 'final' flag for anonymous classes

2016-06-23 Thread Roger Riggs
Hi Dan, On 6/23/2016 2:53 PM, Dan Smith wrote: On Jun 23, 2016, at 11:37 AM, Roger Riggs wrote: Hi Dan, I was concerned about inter-operation between versions because serialized objects are passed between Java runtimes with different versions and both need to compute the same serial version

Re: RFR [9] 8154017: Shutdown hooks are racing against shutdown sequence, if System.exit()-calling thread is interrupted

2016-06-23 Thread Alan Bateman
On 23/06/2016 17:32, Chris Hegarty wrote: The shutdown hooks race against the shutdown sequence if the thread invoking System.exit() is interrupted. This happens because ApplicationShutdownHooks::runHooks joins the shutdown hooks. This may lead to premature VM shutdown, with shutdown hooks not

Re: RFR 8159751: ObjectStreamClass: hide 'final' flag for anonymous classes

2016-06-23 Thread Dan Smith
> On Jun 23, 2016, at 11:37 AM, Roger Riggs wrote: > > Hi Dan, > > I was concerned about inter-operation between versions because serialized > objects > are passed between Java runtimes with different versions and both need to > compute > the same serial version uid (if it is not explicitly de

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.

JVM (1.8.0_91) crashes in Windows native code

2016-06-23 Thread Vinay Purohit
Hello, I'm not sure if this is the right forum for posting an issue we are experiencing with JRE 1.8.0_92. The JVM crashes in the Windows native code and generates a log (see attachment, hs_err_pid15724.log), but not the usual stack-trace due to uncaught exceptions. The root cause is uncle

Re: RFR 9 7131356 : (props) "No Java runtime present, requesting install" when creating VM from JNI [macosx]

2016-06-23 Thread Brent Christian
On 6/23/16 8:24 AM, Naoto Sato wrote: On 6/22/16 9:48 PM, Brent Christian wrote: On 6/22/16 3:58 PM, Naoto Sato wrote: 2. I think mapping language/script combination to a typical locale is ok to keep the compatibility (e.g., "sr-Latn" to "sr_CS", done by the JRS, right?) In the meantime, I wou

Re: RFR 8159751: ObjectStreamClass: hide 'final' flag for anonymous classes

2016-06-23 Thread Roger Riggs
Hi Dan, I was concerned about inter-operation between versions because serialized objects are passed between Java runtimes with different versions and both need to compute the same serial version uid (if it is not explicitly declared). The older java runtimes will compute the serial version u

Re: RFR 9 7131356 : (props) "No Java runtime present, requesting install" when creating VM from JNI [macosx]

2016-06-23 Thread Naoto Sato
I reviewed webrev.04 thoroughly this time and comments below: - Although the OSX API returns the cases as in the spec, BCP 47 language tags are case insensitive, so it would be better to check [a-z] in line 94/95 as well. - Line 82: Does the API actually returns "zh-Hans_HK"? Underscore ('_'

Re: RFR 8159751: ObjectStreamClass: hide 'final' flag for anonymous classes

2016-06-23 Thread Dan Smith
> On Jun 23, 2016, at 8:51 AM, Roger Riggs wrote: > > Hi Dan, > > Is setting ACC_FINAL for anonymous inner classes a unique feature of the > latest bytecode version? > (javac -target 9) > > Was it always clear in class files with earlier bytecode versions? This is a choice that the compiler m

Re: RFR [9] 8154017: Shutdown hooks are racing against shutdown sequence, if System.exit()-calling thread is interrupted

2016-06-23 Thread Aleksey Shipilev
On 23.06.2016 19:32, Chris Hegarty wrote: > The shutdown hooks race against the shutdown sequence if the thread > invoking System.exit() is interrupted. This happens because > ApplicationShutdownHooks::runHooks joins the shutdown hooks. This > may lead to premature VM shutdown, with shutdown hooks

RFR [9] 8154017: Shutdown hooks are racing against shutdown sequence, if System.exit()-calling thread is interrupted

2016-06-23 Thread Chris Hegarty
The shutdown hooks race against the shutdown sequence if the thread invoking System.exit() is interrupted. This happens because ApplicationShutdownHooks::runHooks joins the shutdown hooks. This may lead to premature VM shutdown, with shutdown hooks not fully executed. http://cr.openjdk.java.net/

Re: RFR: 8150173: JAXBContext.newInstance causes PrivilegedActionException when createContext's declared in absract class extended by discovered JAXB implementation

2016-06-23 Thread Daniel Fuchs
Hi, Here is the new webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8150173/webrev.03/ On 23/06/16 14:55, Daniel Fuchs wrote: if I understand correctly JAXBContextWithSubclassedFactory is supposed to capture the idea of Test9 but it doesn't. [...] Oh - I see, I missed that - I was chasing

Re: RFR 9 7131356 : (props) "No Java runtime present, requesting install" when creating VM from JNI [macosx]

2016-06-23 Thread Naoto Sato
On 6/22/16 9:48 PM, Brent Christian wrote: On 6/22/16 3:58 PM, Naoto Sato wrote: Hi Brent, 1. It seems that the display language in the new code seems to have some problems. I see (in es-419.txt): --- locale[Default|Display|Format].getLanguage () is user.language: it is --- And in fact, displ

Re: RFR: JDK-8031043 ClassValue's backing map should have a smaller initial size

2016-06-23 Thread Peter Levart
Hi, Doug suggested to try another variant of resolving hash collisions in open-addressing table - using a secondary hash for increment value of consecutive probes. Since Java Objects provide only one hashCode() function, "secondary" hash is computed from primary hash by xorShift(hashCode), fo

Re: RFR 8159751: ObjectStreamClass: hide 'final' flag for anonymous classes

2016-06-23 Thread Roger Riggs
Hi Dan, Is setting ACC_FINAL for anonymous inner classes a unique feature of the latest bytecode version? (javac -target 9) Was it always clear in class files with earlier bytecode versions? Thanks, Roger On 6/22/2016 7:05 PM, Dan Smith wrote: Hi, making a change to serialization's computa

Re: RFR(L): 8143211: provide bytecode intrinsics for loop and try/finally executors

2016-06-23 Thread Michael Haupt
Dear all, thank you, Claes, Vladimir, and Paul, for your reviews. See http://cr.openjdk.java.net/~mhaupt/8143211/webrev.01/ for the next round. The difference to the previous version is, roughly, as follows. Rev 01 ... ... honours the possibility of there being more than one loop in a LambdaFor

Re: RFR: 8150173: JAXBContext.newInstance causes PrivilegedActionException when createContext's declared in absract class extended by discovered JAXB implementation

2016-06-23 Thread Daniel Fuchs
Hi Georgiy On 23/06/16 13:48, Georgiy Rakov wrote: Hi Daniel, if I understand correctly JAXBContextWithSubclassedFactory is supposed to capture the idea of Test9 but it doesn't. FactoryBase shouldn't implement JAXBContextFactory but Factory should: public static class FactoryBase {

Re: RFR: 8150173: JAXBContext.newInstance causes PrivilegedActionException when createContext's declared in absract class extended by discovered JAXB implementation

2016-06-23 Thread Georgiy Rakov
Hi Daniel, if I understand correctly JAXBContextWithSubclassedFactory is supposed to capture the idea of Test9 but it doesn't. FactoryBase shouldn't implement JAXBContextFactory but Factory should: public static class FactoryBase { public JAXBContext createContext(...) ...

Re: Question about signature polymorphic, the VM spec of Java 7+, and VarHandles

2016-06-23 Thread Peter Levart
On 06/23/2016 02:24 PM, Paul Sandoz wrote: Hi Uwe, Note that javac does not currently use MethodHandle.@PolymorphicSignature, so it only serves as marker for documentation purposes. There may be some unfinished business here, or the unique and limited scope of such methods meant there was,

Re: Question about signature polymorphic, the VM spec of Java 7+, and VarHandles

2016-06-23 Thread Paul Sandoz
Hi Uwe, Note that javac does not currently use MethodHandle.@PolymorphicSignature, so it only serves as marker for documentation purposes. There may be some unfinished business here, or the unique and limited scope of such methods meant there was, contrary to what was originally thought, no nee

RE: Question about signature polymorphic, the VM spec of Java 7+, and VarHandles

2016-06-23 Thread Uwe Schindler
Hi Paul, How about making the used (currently internal) annotation @PolymorphicSignature public in MethodHandle? Then the spec would be much easier to define (and future-proof). Of course, a separate ACC_POLYMORPHIC method access flag would have been better from the beginning! I assume, you had

Re: Question about signature polymorphic, the VM spec of Java 7+, and VarHandles

2016-06-23 Thread Paul Sandoz
Hi Uwe, The JL and JVM specifications have yet to be updated to take into account the new sig-poly methods on VarHandle, they will be in due course by Alex. It’s not clear yet if we can specify this generally or we will have to enumerate on a per-class/method basis. Ideally it should be future