Re: RFR: 8306055: Add a built-in Catalog to JDK XML module

2023-11-19 Thread Alan Bateman
On Sun, 19 Nov 2023 08:44:01 GMT, Alan Bateman wrote: >> Implement the built-in Catalog. > > src/java.xml/share/classes/jdk/xml/internal/jdkcatalog/JDKCatalog.xml line 34: > >> 32: >> 33: http://java.sun.com/dtd/preferences.dtd; >> uri="J2SE/

Re: RFR: JDK-8320168: handle setsocktopt return values [v3]

2023-11-18 Thread Alan Bateman
On Thu, 16 Nov 2023 13:22:10 GMT, Matthias Baesken wrote: >> src/java.base/windows/native/libnio/ch/Net.c line 160: >> >>> 158: >>> 159: /* IPV6_V6ONLY is true by default */ >>> 160: /* attempt to disable IPV6_V6ONLY to ensure dual-socket >>> support; ignore errors */ >> >>

Re: RFR: JDK-8320168: handle setsocktopt return values [v3]

2023-11-18 Thread Alan Bateman
On Thu, 16 Nov 2023 15:52:02 GMT, Matthias Baesken wrote: >> There are a few places in the JDK C coding where the setsocktopt return >> value is not handled but better should be handled. > > Matthias Baesken has updated the pull request incrementally with one > additional commit since the last

Re: RFR: 8320348: java.io.File.getAbsolutePath fails if working directory is not on drive C

2023-11-17 Thread Alan Bateman
On Fri, 17 Nov 2023 21:51:06 GMT, Brian Burkhalter wrote: > The method `windowsDriveRelative` of the test `java/io/File/GetAbsolutePath` > was incorrectly translated from its previous version to the current JUnit 5 > version. The test fix looks okay but the title on the issue is confusing, it

Re: RFR: 8319123: Implement JEP 461: Stream Gatherers (Preview) [v11]

2023-11-17 Thread Alan Bateman
On Fri, 17 Nov 2023 11:28:23 GMT, Viktor Klang wrote: >> This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461) > > Viktor Klang has updated the pull request incrementally with one additional > commit since the last revision: > > Adding @implSpec for Gatherer.andThen, plus adds

Re: RFR: JDK-8320168: handle setsocktopt return values [v3]

2023-11-17 Thread Alan Bateman
On Fri, 17 Nov 2023 08:07:02 GMT, Matthias Baesken wrote: > Hi Alan, do you want an additional change to > src/java.base/windows/native/libnio/ch/WindowsAsynchronousServerSocketChannelImpl.c > ? Was not really sure about this after your comment about this chunk . I think it's okay. The only

Re: RFR: 8318966: Some methods make promises about Java array element alignment that are too strong

2023-11-16 Thread Alan Bateman
On Wed, 15 Nov 2023 22:46:03 GMT, Jorn Vernee wrote: > See the JBS issue for an extended problem description. > > This patch changes the specification and implementation of > `MethodHandles::byteArrayViewVarHandle`, > `MethodHandles::byteBufferViewVarHandle`, `ByteBuffer::alignedSlice`, and

Re: RFE: support safely wrapping restricted FFM calls

2023-11-16 Thread Alan Bateman
On 15/11/2023 17:13, Rob Spoor wrote: Hello all, I'm working on a module that makes working with FFM easier; think of something like JNA. For instance, it allows creating structures without having to manually manage var handles etc. My module uses restricted mehods like

Re: RFR: JDK-8320168: handle setsocktopt return values [v2]

2023-11-16 Thread Alan Bateman
On Thu, 16 Nov 2023 13:14:16 GMT, Matthias Baesken wrote: > Hi Alan, do you mean "drop this chunk from the change" or "drop the coding > that is trying to copy the socket options" ? Yes, I think drop SdpSupport.c from the PR. We probably should have removed the SDP support a long time ago.

Re: RFR: JDK-8319626: Override toString() for ZipFile [v2]

2023-11-16 Thread Alan Bateman
On Wed, 15 Nov 2023 18:33:47 GMT, Justin Lu wrote: >> Please review this PR and [CSR](https://bugs.openjdk.org/browse/JDK-8319982) >> which overrides and provides an implementation of `toString()` in >> _java.util.zip.ZipFile_ (and by extension, _java.util.jar.JarFile_). >> >> This change is

Re: RFR: JDK-8320168: handle setsocktopt return values

2023-11-16 Thread Alan Bateman
On Thu, 16 Nov 2023 08:39:32 GMT, Matthias Baesken wrote: > There are a few places in the JDK C coding where the setsocktopt return value > is not handled but better should be handled. src/java.base/unix/native/libnet/SdpSupport.c line 100: > 98: if (getsockopt(fd, SOL_SOCKET,

Re: RFR: JDK-8320168: handle setsocktopt return values

2023-11-16 Thread Alan Bateman
On Thu, 16 Nov 2023 08:39:32 GMT, Matthias Baesken wrote: > There are a few places in the JDK C coding where the setsocktopt return value > is not handled but better should be handled. src/java.base/windows/native/libnio/ch/Net.c line 160: > 158: > 159: /* IPV6_V6ONLY is true by

Re: return-value check of setsockopt ?

2023-11-16 Thread Alan Bateman
On 16/11/2023 08:40, Baesken, Matthias wrote: Hi Alan, thanks for the comment. I created https://bugs.openjdk.org/browse/JDK-8320168  . Okay, but please note that putting checks into some of these places will require testing across a very wide range of systems. There are at least some

Re: RFR: 8319123: Implement JEP 461: Stream Gatherers (Preview) [v9]

2023-11-15 Thread Alan Bateman
On Wed, 15 Nov 2023 17:50:48 GMT, Viktor Klang wrote: >> This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461) > > Viktor Klang has updated the pull request incrementally with one additional > commit since the last revision: > > Improvements after feedback Really nicely done.

Re: RFR: 8319123: Implement JEP 461: Stream Gatherers (Preview) [v8]

2023-11-15 Thread Alan Bateman
On Tue, 14 Nov 2023 16:35:53 GMT, Viktor Klang wrote: >> This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461) > > Viktor Klang has updated the pull request incrementally with one additional > commit since the last revision: > > Unpacking the rethrown exceptions from

Re: return-value check of setsockopt ?

2023-11-14 Thread Alan Bateman
On 14/11/2023 15:16, Baesken, Matthias wrote: Hi, while looking at some net/nio coding, I noticed  that at a lot of places the return value of setsockopt is checked : https://github.com/search?q=repo%3Aopenjdk%2Fjdk+%22setsockopt%22=code

Re: RFR: 8317834: java/lang/Thread/IsAlive.java timed out [v2]

2023-11-14 Thread Alan Bateman
On Tue, 14 Nov 2023 17:25:42 GMT, Darragh Clarke wrote: >> Currently the `IsAlive` test occasionally times out. >> >> This PR changes the timeout from `10` to `25` which I think is an adequate >> increase based on the failures I've seen. Though I'd be happy to change it >> to another value if

Re: RFR: 8317834: java/lang/Thread/IsAlive.java timed out

2023-11-14 Thread Alan Bateman
On Tue, 14 Nov 2023 16:41:31 GMT, Darragh Clarke wrote: > Currently the `IsAlive` test occasionally times out. > > This PR changes the timeout from `10` to `25` which I think is an adequate > increase based on the failures I've seen. Though I'd be happy to change it to > another value if

Re: RFR: 8314480: Memory ordering spec updates in java.lang.ref

2023-11-14 Thread Alan Bateman
On Mon, 13 Nov 2023 22:31:16 GMT, Brent Christian wrote: > Classes in the `java.lang.ref` package would benefit from an update to bring > the spec in line with how the VM already behaves. The changes would focus on > _happens-before_ edges at some key points during reference processing. > > A

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs [v2]

2023-11-14 Thread Alan Bateman
On Fri, 10 Nov 2023 15:44:19 GMT, Yakov Shafranovich wrote: >> The various Zip/Jar-file related Java APIs have some long-standing >> differences or peculiarities with respect to the ZIP-file specification or >> compared to other implementations which should be documented in the API-doc. >>

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs [v2]

2023-11-14 Thread Alan Bateman
On Fri, 10 Nov 2023 15:44:19 GMT, Yakov Shafranovich wrote: >> The various Zip/Jar-file related Java APIs have some long-standing >> differences or peculiarities with respect to the ZIP-file specification or >> compared to other implementations which should be documented in the API-doc. >>

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs [v2]

2023-11-14 Thread Alan Bateman
On Fri, 10 Nov 2023 15:44:19 GMT, Yakov Shafranovich wrote: >> The various Zip/Jar-file related Java APIs have some long-standing >> differences or peculiarities with respect to the ZIP-file specification or >> compared to other implementations which should be documented in the API-doc. >>

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v6]

2023-11-13 Thread Alan Bateman
On Mon, 13 Nov 2023 15:37:29 GMT, Severin Gehwolf wrote: > It's not clear what you mean by that. Dropped from what? The CSR? Something > else? The run-image based link needs the `--add-run-image-resources` plugin > for it to work. I mean the list of plugins that are listed by --list-plugins.

Re: RFR: JDK-8319626: Override toString() for ZipFile

2023-11-13 Thread Alan Bateman
On Mon, 13 Nov 2023 21:59:05 GMT, Justin Lu wrote: > Please review this PR and [CSR](https://bugs.openjdk.org/browse/JDK-8319982) > which overrides and provides an implementation of `toString()` in > _java.util.zip.ZipFile_ (and by extension, _java.util.jar.JarFile_). > > This change is

Re: RFR: 8287843: File::getCanonicalFile doesn't work for \\?\C:\ style paths DOS device paths [v8]

2023-11-13 Thread Alan Bateman
On Mon, 13 Nov 2023 19:41:48 GMT, Brian Burkhalter wrote: >> In the Windows implementation of java.io.File.getCanonicalPath, strip any >> long path or UNC prefix before canonicalizing the remainder of the pathname. > > Brian Burkhalter has updated the pull request with a new target base due to

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs [v2]

2023-11-13 Thread Alan Bateman
On Mon, 13 Nov 2023 16:00:05 GMT, Yakov Shafranovich wrote: > That would probably also involve taking existing documentation such as the > note about not opening entries with "."/"..", and the POSIX permissions > mappings? Would it make sense to split the rest of the changes in this PR > from

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs [v2]

2023-11-13 Thread Alan Bateman
On Mon, 13 Nov 2023 16:00:05 GMT, Yakov Shafranovich wrote: > That would probably also involve taking existing documentation such as the > note about not opening entries with "."/"..", and the POSIX permissions > mappings? Would it make sense to split the rest of the changes in this PR > from

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs [v2]

2023-11-13 Thread Alan Bateman
On Mon, 13 Nov 2023 16:00:05 GMT, Yakov Shafranovich wrote: > That would probably also involve taking existing documentation such as the > note about not opening entries with "."/"..", and the POSIX permissions > mappings? Would it make sense to split the rest of the changes in this PR > from

Re: RFR: JDK-8317799 : AIX PPC64: FFI symbol lookup doesn't find symbols

2023-11-12 Thread Alan Bateman
On Fri, 10 Nov 2023 16:30:35 GMT, suchismith1993 wrote: > There is not generic way of generating this. It needs a manual intervention > and symbols are to be added on a need basis in future. Looks like a list to > be maintained. I had tried adding comments to explain the list, but that >

Re: RFR: JDK-8317799 : AIX PPC64: FFI symbol lookup doesn't find symbols

2023-11-12 Thread Alan Bateman
On Fri, 10 Nov 2023 16:30:35 GMT, suchismith1993 wrote: > There is not generic way of generating this. It needs a manual intervention > and symbols are to be added on a need basis in future. Looks like a list to > be maintained. I had tried adding comments to explain the list, but that >

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v6]

2023-11-12 Thread Alan Bateman
On Wed, 18 Oct 2023 17:37:30 GMT, Severin Gehwolf wrote: >> Please review this patch which adds a "jmodless" jlink mode to the JDK. >> Fundamentally this patch adds an option to use `jlink` even though your JDK >> install might not come with the packaged modules (directory `jmods`). This >>

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs [v2]

2023-11-11 Thread Alan Bateman
On Fri, 10 Nov 2023 15:44:19 GMT, Yakov Shafranovich wrote: >> The various Zip/Jar-file related Java APIs have some long-standing >> differences or peculiarities with respect to the ZIP-file specification or >> compared to other implementations which should be documented in the API-doc. >>

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs [v2]

2023-11-11 Thread Alan Bateman
On Fri, 10 Nov 2023 15:44:19 GMT, Yakov Shafranovich wrote: >> The various Zip/Jar-file related Java APIs have some long-standing >> differences or peculiarities with respect to the ZIP-file specification or >> compared to other implementations which should be documented in the API-doc. >>

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs [v2]

2023-11-11 Thread Alan Bateman
On Fri, 10 Nov 2023 15:44:19 GMT, Yakov Shafranovich wrote: >> The various Zip/Jar-file related Java APIs have some long-standing >> differences or peculiarities with respect to the ZIP-file specification or >> compared to other implementations which should be documented in the API-doc. >>

Re: RFR: 8287843: File::getCanonicalFile doesn't work for \\?\C:\ style paths DOS device paths [v7]

2023-11-11 Thread Alan Bateman
On Wed, 4 Oct 2023 20:35:25 GMT, Brian Burkhalter wrote: >> In the Windows implementation of java.io.File.getCanonicalPath, strip any >> long path or UNC prefix before canonicalizing the remainder of the pathname. > > Brian Burkhalter has updated the pull request incrementally with one >

Re: RFR: 8319928: Exceptions thrown by cleanup actions passed to reinterpreted segments should be ignored

2023-11-10 Thread Alan Bateman
On Fri, 10 Nov 2023 16:34:11 GMT, Maurizio Cimadamore wrote: > This simplePR tweaks the factory which wraps custom cleanup actions passed to > `MemorySegment::reinterpret`, so that any exception thrown by the custom > cleanup is swallowed when the arena is closed. > > This aligns the

Re: RFR: JDK-8319123 : Implementation of JEP-461: Stream Gatherers (Preview) [v3]

2023-11-10 Thread Alan Bateman
On Fri, 10 Nov 2023 09:32:17 GMT, Viktor Klang wrote: >> This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461) > > Viktor Klang has updated the pull request incrementally with two additional > commits since the last revision: > > - Multiple improvements to Gatherer and

Re: Suggestion needed to port the fix to JDK17 and JDK11

2023-11-10 Thread Alan Bateman
On 10/11/2023 07:26, Shruthi Shruthi1 wrote: Hi Alan., Please let us know if you have any suggestions/inputs for the earlier update. Sorry, I can't help you. The SAP and IBM engineers maintain the AIX port and may be able to help diagnose this issue, it sounds like dup2 is problematic.

Re: RFR: 8284890: Support for Do not fragment IP socket options [v8]

2023-11-09 Thread Alan Bateman
On Wed, 20 Apr 2022 14:30:21 GMT, Michael McMahon wrote: >> Hi, >> >> Could I get the following PR review please? It adds a new JDK specific >> extended socket option >> called IP_DONTFRAGMENT, which disables IP packet fragmentation in both IPv4 >> and IPv6 >> UDP sockets (NIO

Re: RFR: 8284890: Support for Do not fragment IP socket options [v8]

2023-11-09 Thread Alan Bateman
On Wed, 20 Apr 2022 14:30:21 GMT, Michael McMahon wrote: >> Hi, >> >> Could I get the following PR review please? It adds a new JDK specific >> extended socket option >> called IP_DONTFRAGMENT, which disables IP packet fragmentation in both IPv4 >> and IPv6 >> UDP sockets (NIO

Re: RFR: JDK-8319123 : Implementation of JEP-461: Stream Gatherers (Preview) [v2]

2023-11-09 Thread Alan Bateman
On Wed, 8 Nov 2023 21:08:03 GMT, Viktor Klang wrote: >> This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461) > > Viktor Klang has updated the pull request incrementally with two additional > commits since the last revision: > > - Addressing review feedback > - Make

Re: RFR: JDK-8319123 : Implementation of JEP-461: Stream Gatherers (Preview) [v2]

2023-11-09 Thread Alan Bateman
On Thu, 9 Nov 2023 09:42:41 GMT, Viktor Klang wrote: >> src/java.base/share/classes/java/util/stream/Gatherers.java line 326: >> >>> 324: >>> 325: /** >>> 326: * Gathers elements into fixed-size windows. The last window may >>> contain >> >> Returns a ... > > You mean "Returns a

Re: RFR: JDK-8319123 : Implementation of JEP-461: Stream Gatherers (Preview)

2023-11-08 Thread Alan Bateman
On Mon, 30 Oct 2023 15:38:35 GMT, Viktor Klang wrote: > This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461) src/java.base/share/classes/java/util/stream/Gatherer.java line 38: > 36: /** > 37: * An intermediate operation that transforms a stream of input elements > into a >

Re: RFR: 8318971: jar v17 should either exit on error immediately or create archive as jar v1.8 did [v2]

2023-11-08 Thread Alan Bateman
On Wed, 8 Nov 2023 16:03:22 GMT, Ryan Wallace wrote: > Added and tested and it still works as expected. @ryawalla Would it be possible to update the PR description so there's a clear summary on the updated proposal? I think it aligns the behavior with what happens when a bad file path is

Re: RFR: 8319338: tools/jpackage/share/RuntimeImageTest.java fails with -XX:+UseZGC

2023-11-07 Thread Alan Bateman
On Tue, 7 Nov 2023 03:20:45 GMT, Alexey Semenyuk wrote: > Remove `-Xmx512m` from the jtreg `@run` command as @AlanBateman suggested Marked as reviewed by alanb (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/16535#pullrequestreview-1718248698

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v14]

2023-11-07 Thread Alan Bateman
On Tue, 7 Nov 2023 15:45:02 GMT, Alan Bateman wrote: >> Jim Laskey 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 14 addi

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v14]

2023-11-07 Thread Alan Bateman
On Tue, 7 Nov 2023 12:29:12 GMT, Jim Laskey wrote: >> Address changes from JEP 445 to JEP 463. >> >> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class. >> >> - Don't mark class on read. >> >> - Remove reflection and annotation processing related to unnamed classes. >> >> -

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v14]

2023-11-07 Thread Alan Bateman
On Tue, 7 Nov 2023 15:45:02 GMT, Alan Bateman wrote: >> Jim Laskey 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 14 addi

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v14]

2023-11-07 Thread Alan Bateman
On Tue, 7 Nov 2023 12:29:12 GMT, Jim Laskey wrote: >> Address changes from JEP 445 to JEP 463. >> >> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class. >> >> - Don't mark class on read. >> >> - Remove reflection and annotation processing related to unnamed classes. >> >> -

Re: RFR: 8319374: JFR: Remove instrumentation for exception events [v3]

2023-11-07 Thread Alan Bateman
On Tue, 7 Nov 2023 11:11:31 GMT, Erik Gahlin wrote: >> Could I have a review of a PR that removes the bytecode instrumentation for >> the exception events. >> >> Testing: jdk/jdk/jfr + tier1 + tier2 > > Erik Gahlin has updated the pull request incrementally with one additional > commit since

Re: RFR: 8319374: JFR: Remove instrumentation for exception events [v3]

2023-11-07 Thread Alan Bateman
On Tue, 7 Nov 2023 11:11:31 GMT, Erik Gahlin wrote: >> Could I have a review of a PR that removes the bytecode instrumentation for >> the exception events. >> >> Testing: jdk/jdk/jfr + tier1 + tier2 > > Erik Gahlin has updated the pull request incrementally with one additional > commit since

Re: RFR: 8319374: JFR: Remove instrumentation for exception events [v2]

2023-11-07 Thread Alan Bateman
On Tue, 7 Nov 2023 02:10:29 GMT, Erik Gahlin wrote: > I filed an issue to investigate if there is a problem with SOE, or if the OOM > check is really needed now. https://bugs.openjdk.org/browse/JDK-8319579 > > Regardless of outcome, It would be good to document the results of the >

Re: RFR: 8319374: JFR: Remove instrumentation for exception events [v2]

2023-11-07 Thread Alan Bateman
On Tue, 7 Nov 2023 02:10:29 GMT, Erik Gahlin wrote: > I filed an issue to investigate if there is a problem with SOE, or if the OOM > check is really needed now. https://bugs.openjdk.org/browse/JDK-8319579 > > Regardless of outcome, It would be good to document the results of the >

Re: RFR: 8319374: JFR: Remove instrumentation for exception events [v2]

2023-11-07 Thread Alan Bateman
On Mon, 6 Nov 2023 22:52:50 GMT, Erik Gahlin wrote: >> Could I have a review of a PR that removes the bytecode instrumentation for >> the exception events. >> >> Testing: jdk/jdk/jfr + tier1 + tier2 > > Erik Gahlin has updated the pull request incrementally with one additional > commit since

Re: RFR: 8319374: JFR: Remove instrumentation for exception events [v2]

2023-11-07 Thread Alan Bateman
On Mon, 6 Nov 2023 22:52:50 GMT, Erik Gahlin wrote: >> Could I have a review of a PR that removes the bytecode instrumentation for >> the exception events. >> >> Testing: jdk/jdk/jfr + tier1 + tier2 > > Erik Gahlin has updated the pull request incrementally with one additional > commit since

Re: RFR: 8319436: Proxy.newProxyInstance throws NPE if loader is null and interface not visible from class loader [v2]

2023-11-06 Thread Alan Bateman
On Mon, 6 Nov 2023 20:22:43 GMT, Mandy Chung wrote: >> This is a regression caused by JDK-8302791. IAE should be thrown when an >> interface is not visible to the given class loader but NPE is thrown instead >> when the loader is null. The boot loader has no name and so the fix will >>

Re: RFR: 8319338: tools/jpackage/share/RuntimeImageTest.java fails with -XX:+UseZGC

2023-11-06 Thread Alan Bateman
On Tue, 7 Nov 2023 03:20:45 GMT, Alexey Semenyuk wrote: > Remove `-Xmx512m` from the jtreg `@run` command as @AlanBateman suggested Can you confirm that debug builds are passing with -XX:+UseZGC now? - PR Comment: https://git.openjdk.org/jdk/pull/16535#issuecomment-1797889256

Re: RFR: 8319436: Proxy.newProxyInstance throws NPE if loader is null and interface not visible from class loader

2023-11-06 Thread Alan Bateman
On Mon, 6 Nov 2023 19:12:28 GMT, Mandy Chung wrote: > This is a regression caused by JDK-8302791. IAE should be thrown when an > interface is not visible to the given class loader but NPE is thrown instead > when the loader is null. The boot loader has no name and so the fix will > print

Re: RFR: 8319374: JFR: Remove instrumentation for exception events

2023-11-06 Thread Alan Bateman
On Fri, 3 Nov 2023 12:19:07 GMT, Erik Gahlin wrote: > Could I have a review of a PR that removes the bytecode instrumentation for > the exception events. > > Testing: jdk/jdk/jfr + tier1 + tier2 src/java.base/share/classes/jdk/internal/event/ThrowableTracer.java line 37: > 35: private

Re: RFR: 8319374: JFR: Remove instrumentation for exception events

2023-11-06 Thread Alan Bateman
On Fri, 3 Nov 2023 12:19:07 GMT, Erik Gahlin wrote: > Could I have a review of a PR that removes the bytecode instrumentation for > the exception events. > > Testing: jdk/jdk/jfr + tier1 + tier2 src/java.base/share/classes/jdk/internal/event/ThrowableTracer.java line 37: > 35: private

Re: RFR: 8318971: jar v17 should either exit on error immediately or create archive as jar v1.8 did

2023-11-06 Thread Alan Bateman
On Mon, 30 Oct 2023 16:16:52 GMT, Ryan Wallace wrote: > Hi all, > > Please review this fix for jar tool not producing archive if there is a > missing file supplied. Fix is to throw an exception and exit processing when > a missing file is supplied. Current behaviour will recognise missing

Re: RFR: 8318971: jar v17 should either exit on error immediately or create archive as jar v1.8 did

2023-11-06 Thread Alan Bateman
On Mon, 30 Oct 2023 16:16:52 GMT, Ryan Wallace wrote: > Hi all, > > Please review this fix for jar tool not producing archive if there is a > missing file supplied. Fix is to throw an exception and exit processing when > a missing file is supplied. Current behaviour will recognise missing

Re: RFR: 8318631: GetStackTraceSuspendedStressTest.java failed with: check_jvmti_status: JVMTI function returned error: JVMTI_ERROR_THREAD_NOT_ALIVE (15)

2023-11-05 Thread Alan Bateman
On Mon, 6 Nov 2023 05:34:04 GMT, David Holmes wrote: >> test/hotspot/jtreg/serviceability/jvmti/stress/StackTrace/Suspended/libGetStackTraceSuspendedStress.cpp >> line 125: >> >>> 123: err = jvmti->SetEventNotificationMode(JVMTI_ENABLE, >>> JVMTI_EVENT_SINGLE_STEP, vthread); >>> 124: if

Re: RFR: JDK-8315457 Implementation of String Templates (Second Preview) [v4]

2023-11-04 Thread Alan Bateman
On Fri, 3 Nov 2023 15:29:25 GMT, Jim Laskey wrote: >> Update String Templates for a second preview. With the addition of >> >> - Expression type and throws are determined from the `process` method of the >> processor type and not the processor type. >> >> - Qualified `STR` and `RAW` are

Re: RFR: JDK-8315457 Implementation of String Templates (Second Preview) [v4]

2023-11-04 Thread Alan Bateman
On Mon, 16 Oct 2023 14:31:46 GMT, Jim Laskey wrote: > Wasn't sure about that. Thx. When in doubt, JEP 12. Alex provided good guidance for API authors on how `@since` should be used with preview APIs. - PR Review Comment:

Re: RFR: JDK-8315457 Implementation of String Templates (Second Preview) [v4]

2023-11-04 Thread Alan Bateman
On Mon, 16 Oct 2023 14:31:46 GMT, Jim Laskey wrote: > Wasn't sure about that. Thx. When in doubt, JEP 12. Alex provided good guidance for API authors on how `@since` should be used with preview APIs. - PR Review Comment:

Re: RFR: JDK-8315457 Implementation of String Templates (Second Preview) [v4]

2023-11-04 Thread Alan Bateman
On Fri, 3 Nov 2023 15:29:25 GMT, Jim Laskey wrote: >> Update String Templates for a second preview. With the addition of >> >> - Expression type and throws are determined from the `process` method of the >> processor type and not the processor type. >> >> - Qualified `STR` and `RAW` are

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs

2023-11-04 Thread Alan Bateman
On Mon, 30 Oct 2023 17:26:53 GMT, Yakov Shafranovich wrote: > The various Zip/Jar-file related Java APIs have some long-standing > differences or peculiarities with respect to the ZIP-file specification or > compared to other implementations which should be documented in the API-doc. > This

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs

2023-11-04 Thread Alan Bateman
On Mon, 30 Oct 2023 17:26:53 GMT, Yakov Shafranovich wrote: > The various Zip/Jar-file related Java APIs have some long-standing > differences or peculiarities with respect to the ZIP-file specification or > compared to other implementations which should be documented in the API-doc. > This

Re: RFR: JDK-8319122: Improve documentation of various Zip-file related APIs

2023-11-04 Thread Alan Bateman
On Mon, 30 Oct 2023 17:26:53 GMT, Yakov Shafranovich wrote: > The various Zip/Jar-file related Java APIs have some long-standing > differences or peculiarities with respect to the ZIP-file specification or > compared to other implementations which should be documented in the API-doc. > This

Re: RFR: 8318839: Update test thread factory to catch all exceptions [v2]

2023-11-03 Thread Alan Bateman
On Fri, 3 Nov 2023 05:43:38 GMT, David Holmes wrote: > I don't understand what this is trying to do. If any virtual thread has an > uncaught exception then other virtual threads will throw it wrapped in a > RuntimeException. But the first virtual thread that completes (normally or by >

Re: RFR: 8319332: Security properties files inclusion

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 22:20:02 GMT, Martin Balao wrote: >> src/java.base/share/classes/java/security/Security.java line 243: >> >>> 241: if (connection instanceof FileURLConnection >>> fileConnection) { >>> 242: // A local file URL can be interpreted as a Path >>> 243:

Re: RFR: 8319332: Security properties files inclusion

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 22:20:02 GMT, Martin Balao wrote: >> src/java.base/share/classes/java/security/Security.java line 243: >> >>> 241: if (connection instanceof FileURLConnection >>> fileConnection) { >>> 242: // A local file URL can be interpreted as a Path >>> 243:

Re: RFR: JDK-8296240: Augment discussion of test tiers in doc/testing.md [v6]

2023-11-02 Thread Alan Bateman
On Tue, 31 Oct 2023 16:11:47 GMT, Joe Darcy wrote: >> Clarify the intention of tier 1 tests. I'll reflow the paragraph and >> regenerate the HTML file once the wording is agreed upon. > > Joe Darcy has updated the pull request incrementally with one additional > commit since the last revision:

Re: RFR: 8319332: Security properties files inclusion

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 19:07:48 GMT, Francisco Ferrari Bihurriet wrote: > The implementation of this proposal is based on the requirements, > specification and design choices described in the [JDK-8319332] ticket and > its respective CSR [JDK-8319333]. What follows are implementation notes >

Re: RFR: 8319332: Security properties files inclusion

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 19:07:48 GMT, Francisco Ferrari Bihurriet wrote: > The implementation of this proposal is based on the requirements, > specification and design choices described in the [JDK-8319332] ticket and > its respective CSR [JDK-8319333]. What follows are implementation notes >

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v8]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 17:53:12 GMT, Jim Laskey wrote: > I thought it was a clearer way to get rid of the side effect. It means LauncherHelper.checkAndLoadMain description will need to be updated as it says that it checks the public static void main method. But okay, I'll go through what you have

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v8]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 17:53:12 GMT, Jim Laskey wrote: > I thought it was a clearer way to get rid of the side effect. It means LauncherHelper.checkAndLoadMain description will need to be updated as it says that it checks the public static void main method. But okay, I'll go through what you have

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v7]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 15:45:03 GMT, Jim Laskey wrote: > Amazingly enough I seem to have the JNI straight on the first try. Oh, I wasn't expecting that. My comment was suggesting we change LauncherHelper.checkAndLoadMain to set both appClass and mainType after the validate succeeds.

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v7]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 15:45:03 GMT, Jim Laskey wrote: > Amazingly enough I seem to have the JNI straight on the first try. Oh, I wasn't expecting that. My comment was suggesting we change LauncherHelper.checkAndLoadMain to set both appClass and mainType after the validate succeeds.

Re: RFR: 8317620: Build JDK tools with ModuleMainClass attribute [v2]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 16:19:35 GMT, Mandy Chung wrote: >> Tool modules can be created via `jmod --main-class` option such that >> `ModuleMainClass` attribute will be added in `module-info.class` and the >> module's main class can be launched via `java -m ` without >> specifying the name of the

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v4]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 12:33:27 GMT, Jim Laskey wrote: >> Address changes from JEP 445 to JEP 463. >> >> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class. >> >> - Don't mark class on read. >> >> - Remove reflection and annotation processing related to unnamed classes. >> >> -

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v4]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 12:33:27 GMT, Jim Laskey wrote: >> Address changes from JEP 445 to JEP 463. >> >> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class. >> >> - Don't mark class on read. >> >> - Remove reflection and annotation processing related to unnamed classes. >> >> -

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v4]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 12:33:27 GMT, Jim Laskey wrote: >> Address changes from JEP 445 to JEP 463. >> >> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class. >> >> - Don't mark class on read. >> >> - Remove reflection and annotation processing related to unnamed classes. >> >> -

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v4]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 12:33:27 GMT, Jim Laskey wrote: >> Address changes from JEP 445 to JEP 463. >> >> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class. >> >> - Don't mark class on read. >> >> - Remove reflection and annotation processing related to unnamed classes. >> >> -

Re: RFR: JDK-8315921: Invalid CSS declarations in java.lang class documentation [v2]

2023-11-02 Thread Alan Bateman
On Thu, 2 Nov 2023 10:57:15 GMT, Hannes Wallnöfer wrote: >> This change fixes two errors in inline HTML styles in the `java.lang` >> package: >> >> - wrong CSS property name in `java.lang.String` >> - CSS declaration terminated by colon instead of semicolon in >> `java.lang.Thread` >> >>

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v2]

2023-11-02 Thread Alan Bateman
On Wed, 1 Nov 2023 19:14:22 GMT, Jim Laskey wrote: >> Address changes from JEP 445 to JEP 463. >> >> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class. >> >> - Don't mark class on read. >> >> - Remove reflection and annotation processing related to unnamed classes. >> >> -

Re: RFR: JDK-8315458 Implementation of Implicitly Declared Classes and Instance Main Method (Second Preview) [v2]

2023-11-02 Thread Alan Bateman
On Wed, 1 Nov 2023 19:14:22 GMT, Jim Laskey wrote: >> Address changes from JEP 445 to JEP 463. >> >> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class. >> >> - Don't mark class on read. >> >> - Remove reflection and annotation processing related to unnamed classes. >> >> -

Re: RFR: 8317620: Build JDK tools with ModuleMainClass attribute

2023-11-02 Thread Alan Bateman
On Wed, 1 Nov 2023 19:58:07 GMT, Mandy Chung wrote: > Tool modules can be created via `jmod --main-class` option such that > `ModuleMainClass` attribute will be added in `module-info.class` and the > module's main class can be launched via `java -m ` without > specifying the name of the main

Re: RFR: 8316969: Improve CDS module graph support for --module option [v3]

2023-11-01 Thread Alan Bateman
On Tue, 31 Oct 2023 20:45:11 GMT, Calvin Cheung wrote: > Do you prefer the `canArchive` setting be inside `if > (CDS.isDumpingStaticArchive())` like the following? > > ``` > if (CDS.isDumpingStaticArchive()) > canArchive = true; > ``` That's fine. It has been harmless to have

Re: RFR: 8316969: Improve CDS module graph support for --module option [v8]

2023-11-01 Thread Alan Bateman
On Wed, 1 Nov 2023 04:34:35 GMT, Calvin Cheung wrote: >> Please review this changeset for adding support for `--module` (-m) option >> for CDS. >> Changes in the `ModuleBootstrap.java` are needed so that the >> `ArchivedModuleGraph.archive` and `ArchivedBootLayer.archive` are called if >> the

Re: RFR: JDK-8319195: Move tier 1 vector API regression tests to tier 2

2023-10-31 Thread Alan Bateman
On Tue, 31 Oct 2023 20:14:45 GMT, Joe Darcy wrote: > Due to their longer-than-typical running time and because the vector API is > not (yet) in the base module, move the vector API tests in tier 1 to tier 2. jdk_vector_sanity was added by JDK-8295970 to get some coverage in tier1. That

Re: RFR: 8317609: Classfile API fails to verify /jdk.jcmd/sun/tools/jstat/Alignment.class

2023-10-31 Thread Alan Bateman
On Tue, 10 Oct 2023 15:09:41 GMT, Adam Sotona wrote: > JImageValidator has been recently converted to use ClassFile API and original > weak validation of class files was replaced with full class verification. > Unfortunately full class verification is too strong as it includes class >

Re: RFR: 8287843: File::getCanonicalFile doesn't work for \?\C:\ style paths DOS device paths [v7]

2023-10-31 Thread Alan Bateman
On Mon, 16 Oct 2023 16:31:04 GMT, Brian Burkhalter wrote: > The CSR was created. Thanks. I changed the "Compatibility Risk" to medium and expanded its description to more clearly explain the compatibility risk to fix this long standing issue. - PR Comment:

Re: RFR: JDK-8318467 : [jmh] tests concurrent.Queues and concurrent.ProducerConsumer hang with 101+ threads [v2]

2023-10-31 Thread Alan Bateman
On Mon, 30 Oct 2023 15:07:49 GMT, Viktor Klang wrote: >> Discussed with @DougLea and adjusting the queue capacity to at least the >> number of participating threads seems like the most sensible fix. > > Viktor Klang has updated the pull request incrementally with one additional > commit since

Re: RFR: 8316969: Improve CDS module graph support for --module option [v3]

2023-10-31 Thread Alan Bateman
On Tue, 31 Oct 2023 06:17:34 GMT, Calvin Cheung wrote: > I've added the following field in `ArchivedModuleGraph` so that the > `get(String mainModuleName)` will check the `mainModule` before returning > `archivedModuleGraph`. `private static String mainModule;` The `mainModule` > field is

Re: RFR: 8316969: Improve CDS module graph support for --module option [v7]

2023-10-31 Thread Alan Bateman
On Tue, 31 Oct 2023 06:11:50 GMT, Calvin Cheung wrote: >> Please review this changeset for adding support for `--module` (-m) option >> for CDS. >> Changes in the `ModuleBootstrap.java` are needed so that the >> `ArchivedModuleGraph.archive` and `ArchivedBootLayer.archive` are called if >>

Re: RFR: JDK-8319120: Unbound ScopedValue.get() throws the wrong exception [v2]

2023-10-31 Thread Alan Bateman
On Mon, 30 Oct 2023 17:16:54 GMT, Andrew Haley wrote: >> The bug here is a thinko in `ScopedValue.scopedValueBindings()`. >> >> If the JVM runs out of resources, we throw a `VirtualMachineError`. Running >> out of resources can happen at almost any time, and can happen while >>

Re: RFR: JDK-8296240: Augment discussion of test tiers in doc/testing.md [v5]

2023-10-31 Thread Alan Bateman
On Tue, 31 Oct 2023 00:23:44 GMT, Joe Darcy wrote: >> Clarify the intention of tier 1 tests. I'll reflow the paragraph and >> regenerate the HTML file once the wording is agreed upon. > > Joe Darcy has updated the pull request incrementally with one additional > commit since the last revision:

Re: RFR: 8315004: Runtime.halt() debug logging

2023-10-31 Thread Alan Bateman
On Fri, 1 Sep 2023 08:29:41 GMT, Alan Bateman wrote: >> I think you may have missed the comment in the JBS issue. Logging means >> running potentially arbitrary code, doing this at Runtime.halt time is >> problematic. I thought the conclusion from the work on Runtime.exit

<    5   6   7   8   9   10   11   12   13   14   >