Re: RFR: 8132995: Matcher$ImmutableMatchResult should be optimized to reduce space usage [v2]

2023-05-16 Thread Stuart Marks
On Thu, 11 May 2023 09:22:44 GMT, Raffaello Giulietti wrote: >> When appropriate and useful, copies only the relevant portion of the >> `CharSequence` to the match result. > > Raffaello Giulietti has updated the pull request incrementally with one > additional commit since the last revision:

Re: RFR: 8132995: Matcher$ImmutableMatchResult should be optimized to reduce space usage [v2]

2023-05-16 Thread Stuart Marks
On Tue, 16 May 2023 13:38:03 GMT, Claes Redestad wrote: >> Raffaello Giulietti has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Added some randomness in tests. > > src/java.base/share/classes/java/util/regex/Matcher.java line 1381: > >>

Re: RFR: 8306431: File.listRoots method description should be re-examined [v5]

2023-05-16 Thread Nagata-Haruhito
On Tue, 16 May 2023 06:53:49 GMT, Alan Bateman wrote: >> Nagata-Haruhito has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains four additional >>

Re: RFR: 8306431: File.listRoots method description should be re-examined [v6]

2023-05-16 Thread Nagata-Haruhito
> I fixed File.listRoots description. > * remove "the insertion or ejection of removable media" > * change "available" to "existing" > > Please review this change. Nagata-Haruhito has updated the pull request incrementally with three additional commits since the last revision: - Merge branch

Re: RFR: 8287834: Add SymbolLookup::or method [v3]

2023-05-16 Thread John Hendrikx
On Tue, 16 May 2023 22:36:51 GMT, Maurizio Cimadamore wrote: >> This patch adds a simpler method for composing symbol lookups. It is common >> for clients to chain multiple symbol lookups together, e.g. to find a symbol >> in multiple libraries. >> >> A new instance method, namely

Re: JEP 442: Foreign Function & Memory API => why is it again preview API?

2023-05-16 Thread Maurizio Cimadamore
On 16/05/2023 19:03, - wrote: For FFI, I would prefer some parts of the FFI, especially generic ones like Indirect Var Handles, to be promoted from the preview status. They are useful for non-FFI purposes. These parts (e.g. the additional var handle combinators in MethodHandles) are the only

Re: RFR: 8287834: Add SymbolLookup::or method [v3]

2023-05-16 Thread Maurizio Cimadamore
> This patch adds a simpler method for composing symbol lookups. It is common > for clients to chain multiple symbol lookups together, e.g. to find a symbol > in multiple libraries. > > A new instance method, namely `SymbolLookup::or` is added, which first > searches a symbol in the first

RFR: 8308237: add JDWP and JDI virtual thread support for ThreadReference.PopFrames

2023-05-16 Thread Chris Plummer
This is a follow-on to [JDK-8264699](https://bugs.openjdk.org/browse/JDK-8264699) which adds JVMTI PopFrames support for virtual thread. For JDWP and JDI this is mostly a spec update, although JDI needs minor changes to properly throw the correct exception. Note this PR needs JDK-8264699 in

Re: RFR: 8306842: Classfile API performance improvements [v7]

2023-05-16 Thread Claes Redestad
On Tue, 16 May 2023 15:17:13 GMT, Chen Liang wrote: >> src/java.base/share/classes/jdk/internal/classfile/impl/StackMapDecoder.java >> line 83: >> >>> 81: vtis = new >>> VerificationTypeInfo[methodType.parameterCount()]; >>> 82: } >>> 83: for(var arg :

RFR: JDK-8308239: Tighten up accessibility of nested classes in java.lang.invoke

2023-05-16 Thread Joe Darcy
Tightening up accessibility of a few nested classes. There is no practical impact on source compatibility -- source uses within the package still work, but by default the ability to reflectively call methods on these classes is disabled. - Commit messages: - JDK-8308239: Tighten

Integrated: 8308223: failure handler missed jcmd.vm.info command

2023-05-16 Thread Leonid Mesnik
On Tue, 16 May 2023 18:07:16 GMT, Leonid Mesnik wrote: > Trivial fix that added missed useful command. > Tested by running make of failure handler and verifying results. This pull request has now been integrated. Changeset: 563152f3 Author:Leonid Mesnik URL:

Re: RFR: 8308223: failure handler missed jcmd.vm.info command

2023-05-16 Thread Stefan Karlsson
On Tue, 16 May 2023 18:07:16 GMT, Leonid Mesnik wrote: > Trivial fix that added missed useful command. > Tested by running make of failure handler and verifying results. Marked as reviewed by stefank (Reviewer). - PR Review:

Re: RFR: 8308223: failure handler missed jcmd.vm.info command

2023-05-16 Thread Stefan Karlsson
On Tue, 16 May 2023 18:07:16 GMT, Leonid Mesnik wrote: > Trivial fix that added missed useful command. > Tested by running make of failure handler and verifying results. This looks good and "trivial". Thanks for fixing this. - PR Comment:

Re: RFR: JDK-8306112 Implementation of JEP 445: Unnamed Classes and Instance Main Methods (Preview) [v13]

2023-05-16 Thread Alan Bateman
On Mon, 15 May 2023 17:05:48 GMT, Jim Laskey wrote: > InstanceMainTest does check precedence. Are you expecting a test for all > combinations? Sorry, we discussed this in previous comments, ignore my latest comment on this. - PR Review Comment:

Re: RFR: JDK-8306112 Implementation of JEP 445: Unnamed Classes and Instance Main Methods (Preview) [v15]

2023-05-16 Thread Jim Laskey
> Add flexible main methods and anonymous main classes to the Java language. Jim Laskey has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 29 commits: - Merge branch 'master' into 8306112 - Requested Changes #2 - Update

Re: RFR: 8287834: Add SymbolLookup::or method [v2]

2023-05-16 Thread Paul Sandoz
On Tue, 16 May 2023 17:06:02 GMT, Maurizio Cimadamore wrote: >> This patch adds a simpler method for composing symbol lookups. It is common >> for clients to chain multiple symbol lookups together, e.g. to find a symbol >> in multiple libraries. >> >> A new instance method, namely

Re: RFR: 8307326: Package jdk.internal.classfile.java.lang.constant become obsolete

2023-05-16 Thread Chen Liang
On Mon, 15 May 2023 08:38:54 GMT, Adam Sotona wrote: > Package `jdk.internal.classfile.java.lang.constant` containing `ModuleDesc` > and `PackageDesc` become obsolete after > [JDK-8306729](https://bugs.openjdk.org/browse/JDK-8306729). > All references to

RFR: 8308223: failure handler missed jcmd.vm.info command

2023-05-16 Thread Leonid Mesnik
Trivial fix that added missed useful command. Tested by running make of failure handler and verifying results. - Commit messages: - 8308223 Changes: https://git.openjdk.org/jdk/pull/14018/files Webrev: https://webrevs.openjdk.org/?repo=jdk=14018=00 Issue:

Re: RFR: 8306457: Classfile API components implementations should not be exposed [v2]

2023-05-16 Thread Chen Liang
On Sat, 13 May 2023 14:11:11 GMT, Chen Liang wrote: >> The jdk.internal.classfile.components package's interfaces have >> implementations in their nested classes, which are implicitly public and >> exported with the package. They are now moved to the impl package to avoid >> unwanted

Re: RFR: 8306457: Classfile API components implementations should not be exposed [v3]

2023-05-16 Thread Chen Liang
> The jdk.internal.classfile.components package's interfaces have > implementations in their nested classes, which are implicitly public and > exported with the package. They are now moved to the impl package to avoid > unwanted exposures. Fixed a few minor javadoc issues in the interfaces as

Integrated: 8307547: Support variant collations

2023-05-16 Thread Naoto Sato
On Wed, 10 May 2023 20:11:09 GMT, Naoto Sato wrote: > The fix to https://bugs.openjdk.org/browse/JDK-8306927 switched the default > collation for Swedish to the modern one. In order to provide a means for > users who need the old collation, this PR intends to make `Collator` > recognize the

Re: JEP 442: Foreign Function & Memory API => why is it again preview API?

2023-05-16 Thread -
For FFI, I would prefer some parts of the FFI, especially generic ones like Indirect Var Handles, to be promoted from the preview status. They are useful for non-FFI purposes. Has FFI API considered promoting APIs out of preview incrementally, like how Virtual Threads and Scoped values are? Chen

Re: The introduction of Sequenced collections is not a source compatible change

2023-05-16 Thread Alan Snyder
I have not seen any explanation of how this issue is going to be resolved. Given the new information, shouldn’t the CSR review be reopened to re-evaluate the cost/benefit analysis?

Re: JEP 442: Foreign Function & Memory API => why is it again preview API?

2023-05-16 Thread Remi Forax
- Original Message - > From: "Uwe Schindler" > To: "core-libs-dev" > Sent: Tuesday, May 16, 2023 5:38:32 PM > Subject: JEP 442: Foreign Function & Memory API => why is it again preview > API? > Hi, Hi Uwe, > > yesterday Apache Lucene got the information that JDK 21 got the project >

Re: JEP 442: Foreign Function & Memory API => why is it again preview API?

2023-05-16 Thread Uwe Schindler
Hi Alan, Am 16.05.2023 um 18:07 schrieb Alan Bateman: On 16/05/2023 16:38, Uwe Schindler wrote: : Is it really needed to have a next round of preview API? Why not have Java 21 with Foreign API finalized in the same way like virtual threads? To me it is strange that an API like virtual

Re: RFR: 8308046: Move Solaris related Japanese charsets from java.base to jdk.charsets module [v2]

2023-05-16 Thread Naoto Sato
On Tue, 16 May 2023 15:14:03 GMT, Ichiroh Takiguchi wrote: >> According to "JDK 20 Internationalization Guide" >> https://docs.oracle.com/en/java/javase/20/intl/supported-encodings.html >> Following Solaris related Japanese charsets are in "contained in >> jdk.charsets module" list. >> >> -

Integrated: 8300794: Use @snippet in java.util:i18n

2023-05-16 Thread Justin Lu
On Wed, 10 May 2023 23:54:37 GMT, Justin Lu wrote: > Please review this javadoc only change which uses `@snippet` and `@linkplain` > in i18n related util packages. This pull request has now been integrated. Changeset: 4e929918 Author:Justin Lu URL:

Re: RFR: 8287834: Add SymbolLookup::or method [v2]

2023-05-16 Thread Maurizio Cimadamore
> This patch adds a simpler method for composing symbol lookups. It is common > for clients to chain multiple symbol lookups together, e.g. to find a symbol > in multiple libraries. > > A new instance method, namely `SymbolLookup::or` is added, which first > searches a symbol in the first

Re: RFR: 8287834: Add SymbolLookup::or method [v2]

2023-05-16 Thread Maurizio Cimadamore
On Tue, 16 May 2023 16:55:21 GMT, Paul Sandoz wrote: >> I was trying to split the sentence so that we could get a compact javadoc >> header for the factory. How much important is that? > > Not very IHMO. I was just trying to say the same thing you said with less > words, which makes it more

Re: RFR: 8287834: Add SymbolLookup::or method

2023-05-16 Thread Paul Sandoz
On Tue, 16 May 2023 16:50:34 GMT, Maurizio Cimadamore wrote: >> src/java.base/share/classes/java/lang/foreign/SymbolLookup.java line 136: >> >>> 134: * lookup} More specifically, if a symbol is not found using this >>> lookup, the provided lookup will be >>> 135: * used as fallback.

Re: RFR: 8287834: Add SymbolLookup::or method

2023-05-16 Thread Paul Sandoz
On Fri, 12 May 2023 12:11:23 GMT, Maurizio Cimadamore wrote: > This patch adds a simpler method for composing symbol lookups. It is common > for clients to chain multiple symbol lookups together, e.g. to find a symbol > in multiple libraries. > > A new instance method, namely

Re: RFR: 8287834: Add SymbolLookup::or method

2023-05-16 Thread Maurizio Cimadamore
On Tue, 16 May 2023 16:48:13 GMT, Paul Sandoz wrote: >> This patch adds a simpler method for composing symbol lookups. It is common >> for clients to chain multiple symbol lookups together, e.g. to find a symbol >> in multiple libraries. >> >> A new instance method, namely `SymbolLookup::or`

Re: RFR: 8308016: Use snippets in java.io package [v5]

2023-05-16 Thread Brian Burkhalter
> Replace `{@code ...}` patterns and the like with `{@snippet > lang=java : ...}`. Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision: 8308016: Reduce linking in File::toPath snippet - Changes: - all:

Re: JEP 442: Foreign Function & Memory API => why is it again preview API?

2023-05-16 Thread Maurizio Cimadamore
On 16/05/2023 16:38, Uwe Schindler wrote: It exists now since Java 14, where it first appeared as incubation. I know theres still work on the foreign linker, but why aren't the stable classes like MemorySegment, ValueLayout not public now? Hi Uwe, while I understand the frustration (and I'm

Re: RFR: 8307547: Support variant collations [v5]

2023-05-16 Thread Naoto Sato
> The fix to https://bugs.openjdk.org/browse/JDK-8306927 switched the default > collation for Swedish to the modern one. In order to provide a means for > users who need the old collation, this PR intends to make `Collator` > recognize the `co` Unicode locale extension so that multiple

Re: JEP 442: Foreign Function & Memory API => why is it again preview API?

2023-05-16 Thread Alan Bateman
On 16/05/2023 16:38, Uwe Schindler wrote: : Is it really needed to have a next round of preview API? Why not have Java 21 with Foreign API finalized in the same way like virtual threads? To me it is strange that an API like virtual threads was thrown to end-users (it caused problems at many

Re: RFR: 7065228: To interpret case-insensitive string locale independently

2023-05-16 Thread Naoto Sato
On Tue, 16 May 2023 10:38:52 GMT, Darragh Clarke wrote: > Updated instances of `toLowerCase` and `toUpperCase` in several net and io > files to specify `Locale.ROOT` to ensure that case conversion issues don't > occur, > > I didn't add any new tests but ran tier 1-3 with no issues LGTM. Nice

JEP 442: Foreign Function & Memory API => why is it again preview API?

2023-05-16 Thread Uwe Schindler
Hi, yesterday Apache Lucene got the information that JDK 21 got the project panama JEP 442 update and I implemented it already in our source tree. Unfortunately the API is again marked "preview", but JDK 21 is "LTS release". Many of our users (Elasticserach, Solr) will be switching to this

Re: RFR: 8306842: Classfile API performance improvements [v7]

2023-05-16 Thread Chen Liang
On Tue, 16 May 2023 14:23:04 GMT, Claes Redestad wrote: >> Adam Sotona has updated the pull request incrementally with one additional >> commit since the last revision: >> >> LinkedList replaced with ArrayList in benchmarks > >

Re: RFR: 8308046: Move Solaris related Japanese charsets from java.base to jdk.charsets module [v2]

2023-05-16 Thread Ichiroh Takiguchi
> According to "JDK 20 Internationalization Guide" > https://docs.oracle.com/en/java/javase/20/intl/supported-encodings.html > Following Solaris related Japanese charsets are in "contained in jdk.charsets > module" list. > > - PCK (x-PCK) > - EUC_JP_Solaris (x-eucJP-Open) > > These are not

Re: RFR: 8306842: Classfile API performance improvements [v7]

2023-05-16 Thread Claes Redestad
On Tue, 16 May 2023 08:17:02 GMT, Adam Sotona wrote: >> Following improvements implemented: >> - Switch over `String` replaced with switch single char >> - Binary search for frames in `StackMapGenerator` >> - `StackMapGenerator.rawHandlers` with pre-calculated offsets >> - `ClassEntry` is

Re: RFR: 8307483: New micros for j.u.c.LockSupport [v3]

2023-05-16 Thread Eric Caspole
On Tue, 9 May 2023 19:13:42 GMT, Eric Caspole wrote: >> These micros were developed while investigating JDK-8305670 by myself and >> Sergey Kuksenko. The order of thread creation was important in that bug, so >> there are 2 JMH for creating sleepers before and after the worker threads. > >

Re: RFR: 8132995: Matcher$ImmutableMatchResult should be optimized to reduce space usage [v2]

2023-05-16 Thread Claes Redestad
On Thu, 11 May 2023 09:22:44 GMT, Raffaello Giulietti wrote: >> When appropriate and useful, copies only the relevant portion of the >> `CharSequence` to the match result. > > Raffaello Giulietti has updated the pull request incrementally with one > additional commit since the last revision:

Re: RFR: 7065228: To interpret case-insensitive string locale independently

2023-05-16 Thread Daniel Fuchs
On Tue, 16 May 2023 10:38:52 GMT, Darragh Clarke wrote: > Updated instances of `toLowerCase` and `toUpperCase` in several net and io > files to specify `Locale.ROOT` to ensure that case conversion issues don't > occur, > > I didn't add any new tests but ran tier 1-3 with no issues Looks

Re: RFR: 8307990: jspawnhelper must close its writing side of a pipe before reading from it

2023-05-16 Thread Thomas Stuefe
On Tue, 16 May 2023 12:07:10 GMT, David Holmes wrote: > > > I wonder if @Martin-Buchholz is able to look at this one? > > > My concern with changes like this is that they fix an issue but then have > > > unexpected side-effects themselves. > > > > > > Are there any specific concerns you have?

Re: RFR: 8307990: jspawnhelper must close its writing side of a pipe before reading from it

2023-05-16 Thread David Holmes
On Tue, 16 May 2023 07:56:35 GMT, Thomas Stuefe wrote: > > I wonder if @Martin-Buchholz is able to look at this one? > > My concern with changes like this is that they fix an issue but then have > > unexpected side-effects themselves. > > Are there any specific concerns you have? There seems

RFR: 7065228: To interpret case-insensitive string locale independently

2023-05-16 Thread Darragh Clarke
Updated instances of `toLowerCase` and `toUpperCase` in several net and io files to specify `Locale.ROOT` to ensure that case conversion issues don't occur, I didn't add any new tests but ran tier 1-3 with no issues - Commit messages: - added some case conversions missed

Re: RFR: 8307990: jspawnhelper must close its writing side of a pipe before reading from it

2023-05-16 Thread Thomas Stuefe
On Mon, 15 May 2023 16:11:46 GMT, Volker Simonis wrote: > > 2. I think you don't actually have to hand in the in-pipe-read-end fd > > number via command line arg, just to have the child to close it. You could > > just, in the parent, set the fd to FD_CLOEXEC. Since posix_spawn() exec's > >

Re: RFR: 8306842: Classfile API performance improvements [v5]

2023-05-16 Thread Adam Sotona
On Mon, 15 May 2023 19:06:16 GMT, Glavo wrote: >> Adam Sotona has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - changed LinkedList to ArrayList in RebuildMethodBodies benchmark >> - added RepeatedModelTraversal JMH benchmark > >

Re: RFR: 8306842: Classfile API performance improvements [v7]

2023-05-16 Thread Adam Sotona
> Following improvements implemented: > - Switch over `String` replaced with switch single char > - Binary search for frames in `StackMapGenerator` > - `StackMapGenerator.rawHandlers` with pre-calculated offsets > - `ClassEntry` is caching `ClassDesc` symbol > - Caching of type symbols in

Re: RFR: 8308016: Use snippets in java.io package [v4]

2023-05-16 Thread Alan Bateman
On Mon, 15 May 2023 20:48:47 GMT, Brian Burkhalter wrote: >> Replace `{@code ...}` patterns and the like with `{@snippet >> lang=java : ...}`. > > Brian Burkhalter has updated the pull request incrementally with one > additional commit since the last revision: > > 8308016: Fix link in

Re: RFR: 8307990: jspawnhelper must close its writing side of a pipe before reading from it

2023-05-16 Thread Thomas Stuefe
On Tue, 16 May 2023 07:24:31 GMT, David Holmes wrote: > I wonder if @Martin-Buchholz is able to look at this one? > > My concern with changes like this is that they fix an issue but then have > unexpected side-effects themselves. Are there any specific concerns you have? - PR

Re: RFR: 8307990: jspawnhelper must close its writing side of a pipe before reading from it

2023-05-16 Thread Thomas Stuefe
On Mon, 15 May 2023 18:45:24 GMT, Roger Riggs wrote: >> Since JDK13, executing commands in a sub-process defaults to the so called >> `POSIX_SPAWN` launching mechanism (i.e. >> `-Djdk.lang.Process.launchMechanism=POSIX_SPAWN`) on Linux. This works by >> using `posix_spawn(3)` to firstly start

Re: RFR: 8306842: Classfile API performance improvements [v4]

2023-05-16 Thread Adam Sotona
On Mon, 15 May 2023 17:04:54 GMT, Claes Redestad wrote: >> I'm not questioning performance differences between list implementations. >> The implementation of top level list for this benchmark is irrelevant >> because ~10ns difference cannot affect ~100µs benchmark run. > > Fair point. The only

Re: RFR: 8306842: Classfile API performance improvements [v6]

2023-05-16 Thread Adam Sotona
> Following improvements implemented: > - Switch over `String` replaced with switch single char > - Binary search for frames in `StackMapGenerator` > - `StackMapGenerator.rawHandlers` with pre-calculated offsets > - `ClassEntry` is caching `ClassDesc` symbol > - Caching of type symbols in

Re: RFR: 8307990: jspawnhelper must close its writing side of a pipe before reading from it

2023-05-16 Thread David Holmes
On Fri, 12 May 2023 15:24:19 GMT, Volker Simonis wrote: > Since JDK13, executing commands in a sub-process defaults to the so called > `POSIX_SPAWN` launching mechanism (i.e. > `-Djdk.lang.Process.launchMechanism=POSIX_SPAWN`) on Linux. This works by > using `posix_spawn(3)` to firstly start

Re: RFR: 8306842: Classfile API performance improvements [v5]

2023-05-16 Thread Andrey Turbanov
On Mon, 15 May 2023 14:48:58 GMT, Adam Sotona wrote: >> Following improvements implemented: >> - Switch over `String` replaced with switch single char >> - Binary search for frames in `StackMapGenerator` >> - `StackMapGenerator.rawHandlers` with pre-calculated offsets >> - `ClassEntry` is

Re: RFR: 8306431: File.listRoots method description should be re-examined [v5]

2023-05-16 Thread Alan Bateman
On Tue, 16 May 2023 01:34:45 GMT, Nagata-Haruhito wrote: >> I fixed File.listRoots description. >> * remove "the insertion or ejection of removable media" >> * change "available" to "existing" >> >> Please review this change. > > Nagata-Haruhito has updated the pull request with a new target

Re: RFR: 8306431: File.listRoots method description should be re-examined [v5]

2023-05-16 Thread Nagata-Haruhito
On Tue, 16 May 2023 01:34:45 GMT, Nagata-Haruhito wrote: >> I fixed File.listRoots description. >> * remove "the insertion or ejection of removable media" >> * change "available" to "existing" >> >> Please review this change. > > Nagata-Haruhito has updated the pull request with a new target