Re: RFR: JDK-8318761: MessageFormat pattern support for CompactNumberFormat, ListFormat, and DateTimeFormatter [v8]

2024-02-09 Thread Justin Lu
> Please review this PR and [CSR](https://bugs.openjdk.org/browse/JDK-8319344) > which adds MessageFormat pattern support for the following subformats: > ListFormat, CompactNumberFormat, and DateTimeFormatter. This change is > intended to provide pattern support for the more recently added JDK

RFR: 8325579: Inconsistent behavior in com.sun.jndi.ldap.Connection::createSocket

2024-02-09 Thread Christoph Langer
During analysing a customer case I figured out that we have an inconsistency between documentation and actual behavior in class com.sun.jndi.ldap.Connection. The [method documentation of

Re: RFR: JDK-8318761: MessageFormat pattern support for CompactNumberFormat, ListFormat, and DateTimeFormatter [v7]

2024-02-09 Thread Roger Riggs
On Thu, 8 Feb 2024 21:29:19 GMT, Justin Lu wrote: >> Please review this PR and [CSR](https://bugs.openjdk.org/browse/JDK-8319344) >> which adds MessageFormat pattern support for the following subformats: >> ListFormat, CompactNumberFormat, and DateTimeFormatter. This change is >> intended to

Re: RFR: 8325558: Add jcheck whitespace checking for properties files

2024-02-09 Thread Naoto Sato
On Fri, 9 Feb 2024 13:35:55 GMT, Magnus Ihse Bursie wrote: > This is an attempt to finally implement the idea brought forward in > JDK-8295729: Properties files is essentially source code. It should have the > same whitespace checks as all other source code, so we don't get spurious >

Re: RFR: 8303891: Speed up Zip64SizeTest using a small ZIP64 file [v8]

2024-02-09 Thread Lance Andersen
On Fri, 9 Feb 2024 12:31:27 GMT, Eirik Bjørsnøs wrote: >> Please review this PR which suggests we speed up the `Zip64SizeTest` using a >> small-sized ZIP64 ZIP file specifically created to reproduce the issue being >> tested. >> >> The disk space requirement of this test is known to cause

Integrated: 8325203: System.exit(0) kills the launched 3rd party application

2024-02-09 Thread Alexey Semenyuk
On Thu, 8 Feb 2024 20:57:51 GMT, Alexey Semenyuk wrote: > Tested with the use case from the CR. > > The idea of the fix is to prevent grandchildren processes from being > automatically attached to the job killing all processes attached to it when > the job object is destroyed. > > Filed a

Re: RFR: 8325558: Add jcheck whitespace checking for properties files

2024-02-09 Thread Erik Joelsson
On Fri, 9 Feb 2024 13:42:02 GMT, Magnus Ihse Bursie wrote: >> This is an attempt to finally implement the idea brought forward in >> JDK-8295729: Properties files is essentially source code. It should have >> the same whitespace checks as all other source code, so we don't get >> spurious

Re: RFR: 8325558: Add jcheck whitespace checking for properties files

2024-02-09 Thread Daniel Fuchs
On Fri, 9 Feb 2024 13:35:55 GMT, Magnus Ihse Bursie wrote: > This is an attempt to finally implement the idea brought forward in > JDK-8295729: Properties files is essentially source code. It should have the > same whitespace checks as all other source code, so we don't get spurious >

Re: RFR: 8324573: HashMap::putAll should resize to sum of both map sizes [v4]

2024-02-09 Thread Volker Simonis
On Fri, 2 Feb 2024 17:38:13 GMT, Joshua Cao wrote: >> This change mirrors what we did for ConcurrentHashMap in >> https://github.com/openjdk/jdk/pull/17116. When we add all entries from one >> map to anther, we should resize that map to the size of the sum of both maps. >> >> I used the

Re: RFR: 8325558: Add jcheck whitespace checking for properties files

2024-02-09 Thread Magnus Ihse Bursie
On Fri, 9 Feb 2024 13:35:55 GMT, Magnus Ihse Bursie wrote: > This is an attempt to finally implement the idea brought forward in > JDK-8295729: Properties files is essentially source code. It should have the > same whitespace checks as all other source code, so we don't get spurious >

Re: RFR: 8325558: Add jcheck whitespace checking for properties files

2024-02-09 Thread Magnus Ihse Bursie
On Fri, 9 Feb 2024 13:35:55 GMT, Magnus Ihse Bursie wrote: > This is an attempt to finally implement the idea brought forward in > JDK-8295729: Properties files is essentially source code. It should have the > same whitespace checks as all other source code, so we don't get spurious >

RFR: 8325558: Add jcheck whitespace checking for properties files

2024-02-09 Thread Magnus Ihse Bursie
This is an attempt to finally implement the idea brought forward in JDK-8295729: Properties files is essentially source code. It should have the same whitespace checks as all other source code, so we don't get spurious trailing whitespace changes or leading tabs instead of spaces. With Skara

Re: RFR: 8303891: Speed up Zip64SizeTest using a small ZIP64 file [v8]

2024-02-09 Thread Eirik Bjørsnøs
> Please review this PR which suggests we speed up the `Zip64SizeTest` using a > small-sized ZIP64 ZIP file specifically created to reproduce the issue being > tested. > > The disk space requirement of this test is known to cause problems in some > builds, see

Re: RFR: 8303891: Speed up Zip64SizeTest using a small ZIP64 file [v6]

2024-02-09 Thread Eirik Bjørsnøs
On Fri, 9 Feb 2024 11:59:13 GMT, Lance Andersen wrote: > Again minor but perhaps articulate that after this call, the extra data will > be(obvious to you and I but probably not to others...) Added `zipdetails` describing the resulting ZIP64 structure to the `updateCENHeaderToZip64` method

Re: RFR: 8303891: Speed up Zip64SizeTest using a small ZIP64 file [v6]

2024-02-09 Thread Eirik Bjørsnøs
On Fri, 9 Feb 2024 11:55:16 GMT, Lance Andersen wrote: >> Eirik Bjørsnøs has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Spell fix for "Contents" >> - Add comments to explain how we temporarily use the 'unknown' tag to >> prevent

Re: RFR: 8303891: Speed up Zip64SizeTest using a small ZIP64 file [v6]

2024-02-09 Thread Eirik Bjørsnøs
On Fri, 9 Feb 2024 12:17:17 GMT, Eirik Bjørsnøs wrote: >> test/jdk/java/util/zip/ZipFile/Zip64SizeTest.java line 114: >> >>> 112: >>> 113: // Make an extra field with the correct size for an 8-byte >>> 'uncompressed size' >>> 114: // Zip64 field. Temporarily use the

Re: RFR: 8303891: Speed up Zip64SizeTest using a small ZIP64 file [v7]

2024-02-09 Thread Eirik Bjørsnøs
> Please review this PR which suggests we speed up the `Zip64SizeTest` using a > small-sized ZIP64 ZIP file specifically created to reproduce the issue being > tested. > > The disk space requirement of this test is known to cause problems in some > builds, see

Re: RFR: 8303891: Speed up Zip64SizeTest using a small ZIP64 file [v6]

2024-02-09 Thread Lance Andersen
On Fri, 9 Feb 2024 10:33:30 GMT, Eirik Bjørsnøs wrote: >> Please review this PR which suggests we speed up the `Zip64SizeTest` using a >> small-sized ZIP64 ZIP file specifically created to reproduce the issue being >> tested. >> >> The disk space requirement of this test is known to cause

Re: RFR: 8324539: Do not use LFS64 symbols in JDK libs [v10]

2024-02-09 Thread Alan Bateman
On Thu, 8 Feb 2024 07:44:18 GMT, Magnus Ihse Bursie wrote: >> Similar to [JDK-8318696](https://bugs.openjdk.org/browse/JDK-8318696), we >> should use -D_FILE_OFFSET_BITS=64, and not -D_LARGEFILE64_SOURCE in the JDK >> native libraries. > > Magnus Ihse Bursie has updated the pull request

Re: RFR: 8325485: IncrementInstructions.of(int, int) is not storing the args [v2]

2024-02-09 Thread Adam Sotona
> ClassFile API provides two sets of instructions implementations (bound and > unbound). > Unbound implementation of `IncrementInstruction::constant` returns invalid > value. > This bug discovered a hole in the ClassFile API test coverage. > > This patch provides very simple fix of

Re: RFR: 8303891: Speed up Zip64SizeTest using a small ZIP64 file [v5]

2024-02-09 Thread Eirik Bjørsnøs
On Fri, 9 Feb 2024 06:39:31 GMT, Jaikiran Pai wrote: >> Eirik Bjørsnøs 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 13 additional >> commits

Re: RFR: 8303891: Speed up Zip64SizeTest using a small ZIP64 file [v6]

2024-02-09 Thread Eirik Bjørsnøs
> Please review this PR which suggests we speed up the `Zip64SizeTest` using a > small-sized ZIP64 ZIP file specifically created to reproduce the issue being > tested. > > The disk space requirement of this test is known to cause problems in some > builds, see

Re: RFR: 8325485: IncrementInstructions.of(int, int) is not storing the args

2024-02-09 Thread Andrey Turbanov
On Thu, 8 Feb 2024 13:18:41 GMT, Adam Sotona wrote: > ClassFile API provides two sets of instructions implementations (bound and > unbound). > Unbound implementation of `IncrementInstruction::constant` returns invalid > value. > This bug discovered a hole in the ClassFile API test coverage. >