Re: RFR: 8289227: Support for BCP 47 Extension T - Transformed Content [v5]

2022-07-30 Thread Naoto Sato
On Sat, 30 Jul 2022 01:39:54 GMT, Joe Wang wrote: >> Naoto Sato has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Parse invalid fields correctly > > test/jdk/java/util/Locale/bcp47/TExtensionTests.java line 190: > >> 188: } catch

Re: RFR: 8289227: Support for BCP 47 Extension T - Transformed Content [v6]

2022-07-30 Thread Naoto Sato
> This PR is to propose supporting the `T` extension to the BCP 47 to which > `java.util.Locale` class conforms. There are two extensions to the BCP 47, > one is `Unicode Locale Extension` which has been supported since JDK7, the > other is this `Transformed Content` extension. A CSR has also be

Re: RFR: 8283660: Convert com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java finalizer to Cleaner [v16]

2022-07-30 Thread Peter Levart
On Tue, 26 Jul 2022 05:26:31 GMT, Stuart Marks wrote: > Hans Boehm wrote: > > > I also have concerns about the use of fullFence here. I believe it should > > be the case that reachabilityFence guarantees visibility of memory > > operations program-ordered before the reachabilityFence(p) to the

Re: RFR: 8283660: Convert com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java finalizer to Cleaner [v8]

2022-07-30 Thread Peter Levart
On Sat, 30 Jul 2022 11:07:25 GMT, Peter Levart wrote: >> src/java.naming/share/classes/com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java >> line 216: >> >>> 214: } finally { >>> 215: // Ensure Cleaner does not run until after this method >>> completes >>> 216:

Re: RFR: 8283660: Convert com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java finalizer to Cleaner [v16]

2022-07-30 Thread Peter Levart
On Fri, 22 Jul 2022 20:51:59 GMT, Brent Christian wrote: >> Please review this change to replace the finalizer in >> `AbstractLdapNamingEnumeration` with a Cleaner. >> >> The pieces of state required for cleanup (`LdapCtx homeCtx`, `LdapResult >> res`, and `LdapClient enumClnt`) are moved to a

Re: RFR: 8283660: Convert com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java finalizer to Cleaner [v8]

2022-07-30 Thread Peter Levart
On Wed, 1 Jun 2022 21:40:43 GMT, Roger Riggs wrote: > I don't think there is any benefit to the `try{} finally {fence}`. The > reachabilityFence has no executable code. Its only purpose is to keep the > reference in scope alive. ...reachability fence has no code, but puting it into finally blo

Re: RFR: 8291360: Create entry points to expose low-level class file information

2022-07-30 Thread David Holmes
On Sat, 30 Jul 2022 07:44:11 GMT, David Holmes wrote: >> Or, Can the VM do this traversal as/more efficiently than doing at the java >> level? > > It is usual for the Java code to do such checks and throw exceptions, so that > the VM assumes it is correct and only asserts incase something has b

Re: RFR: 8291360: Create entry points to expose low-level class file information

2022-07-30 Thread David Holmes
On Fri, 29 Jul 2022 21:19:27 GMT, Roger Riggs wrote: >> src/hotspot/share/prims/jvm.cpp line 4059: >> >>> 4057: return JVM_CLASSFILE_MAJOR_VERSION; >>> 4058: } >>> 4059: assert(!java_lang_Class::as_Klass(mirror)->is_array_klass(), >>> "unexpected array class"); >> >> Can this throw Ill

RFR: Merge jdk19

2022-07-30 Thread David Holmes
Forward port JDK 19 -> JDK 20 - Commit messages: - Merge remote-tracking branch 'jdk19/master' into Merge_jdk19 - 8291512: Snippetize modules API examples The webrevs contain the adjustments done while merging with regards to each parent branch: - master: https://webrevs.openjdk.