[jdk25] Integrated: 8358809: Improve link to stdin.encoding from java.lang.IO

2025-06-06 Thread Stuart Marks
On Fri, 6 Jun 2025 22:07:38 GMT, Stuart Marks wrote: > Hi all, > > This pull request contains a backport of commit > [d024f58e](https://github.com/openjdk/jdk/commit/d024f58e61ec27f6c13fde5dadb95c31875815d6) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit bein

Re: [jdk25] RFR: 8358809: Improve link to stdin.encoding from java.lang.IO

2025-06-06 Thread Stuart Marks
On Fri, 6 Jun 2025 22:07:38 GMT, Stuart Marks wrote: > Hi all, > > This pull request contains a backport of commit > [d024f58e](https://github.com/openjdk/jdk/commit/d024f58e61ec27f6c13fde5dadb95c31875815d6) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit bein

Re: RFR: 8358731: Remove jdk.internal.access.JavaAWTAccess.java

2025-06-06 Thread Daniel Fuchs
On Fri, 6 Jun 2025 18:35:53 GMT, Phil Race wrote: > The fix for JDK-8344235: Revisit SecurityManager usage in java.logging after > JEP 486 and JEP 491 integration > removed use of AppContext from java/util/logging/LogManager.java. > That was the only place in the JDK that used > jdk.internal.ac

Re: [jdk25] RFR: 8358809: Improve link to stdin.encoding from java.lang.IO

2025-06-06 Thread Naoto Sato
On Fri, 6 Jun 2025 22:07:38 GMT, Stuart Marks wrote: > Hi all, > > This pull request contains a backport of commit > [d024f58e](https://github.com/openjdk/jdk/commit/d024f58e61ec27f6c13fde5dadb95c31875815d6) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit bein

[jdk25] RFR: 8358809: Improve link to stdin.encoding from java.lang.IO

2025-06-06 Thread Stuart Marks
Hi all, This pull request contains a backport of commit [d024f58e](https://github.com/openjdk/jdk/commit/d024f58e61ec27f6c13fde5dadb95c31875815d6) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. The commit being backported was authored by Stuart Marks on 6 Jun 2025 and was revi

Integrated: 8358809: Improve link to stdin.encoding from java.lang.IO

2025-06-06 Thread Stuart Marks
On Fri, 6 Jun 2025 18:34:10 GMT, Stuart Marks wrote: > Use a link of the form `System##stdin.encoding` to link directly to the > system property's description. This pull request has now been integrated. Changeset: d024f58e Author:Stuart Marks URL: https://git.openjdk.org/jdk/commit

Re: RFR: 8358809: better link to stdin.encoding from java.lang.IO

2025-06-06 Thread Naoto Sato
On Fri, 6 Jun 2025 18:34:10 GMT, Stuart Marks wrote: > Use a link of the form `System##stdin.encoding` to link directly to the > system property's description. Marked as reviewed by naoto (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/25676#pullrequestreview-2906011665

Re: RFR: 8352565: Add native method implementation of Reference.get() [v10]

2025-06-06 Thread Vladimir Ivanov
On Fri, 6 Jun 2025 06:00:48 GMT, Kim Barrett wrote: >> Please review this change which adds a native method providing the >> implementation of Reference::get. Referece::get is an intrinsic candidate, >> so >> this native method implementation is only used when the intrinsic is not. >> >> Curre

RFR: 8358731: Remove jdk.internal.access.JavaAWTAccess.java

2025-06-06 Thread Phil Race
The fix for JDK-8344235: Revisit SecurityManager usage in java.logging after JEP 486 and JEP 491 integration removed use of AppContext from java/util/logging/LogManager.java. That was the only place in the JDK that used jdk.internal.access.JavaAWTAccess.java so it can be removed. There are 3 tes

Re: RFR: 8358731: Remove jdk.internal.access.JavaAWTAccess.java

2025-06-06 Thread Phil Race
On Fri, 6 Jun 2025 18:35:53 GMT, Phil Race wrote: > The fix for JDK-8344235: Revisit SecurityManager usage in java.logging after > JEP 486 and JEP 491 integration > removed use of AppContext from java/util/logging/LogManager.java. > That was the only place in the JDK that used > jdk.internal.ac

RFR: 8358809: better link to stdin.encoding from java.lang.IO

2025-06-06 Thread Stuart Marks
Use a link of the form `System##stdin.encoding` to link directly to the system property's description. - Commit messages: - 8358809: better link to stdin.encoding from java.lang.IO Changes: https://git.openjdk.org/jdk/pull/25676/files Webrev: https://webrevs.openjdk.org/?repo=jdk

Re: RFR: 8358426: Improve lazy computation in Locale [v2]

2025-06-06 Thread Naoto Sato
On Thu, 5 Jun 2025 20:29:48 GMT, Justin Lu wrote: >> Please review this PR which improves occurrences of lazy computation in >> `Locale` and `BaseLocale`. >> >> Existing lazy initialization strategies such as CHM, static nested class, >> and local inner class are replaced with Stable Values. >

Re: RFR: 8358626: Emit UTF-8 CLDR resources [v2]

2025-06-06 Thread Naoto Sato
On Fri, 6 Jun 2025 06:57:20 GMT, Volkan Yazici wrote: >> Naoto Sato has updated the pull request incrementally with one additional >> commit since the last revision: >> >> java.base/Gensrc.gmk comment > > make/modules/jdk.localedata/Gensrc.gmk line 49: > >> 47: -o $(GENSRC_DIR) \ >> 48:

Re: RFR: 8358540: Enhance MathUtils in view of FloatingDecimal enhancements [v2]

2025-06-06 Thread Raffaello Giulietti
> Another step in enhancing floating-point <-> decimal conversions. Raffaello Giulietti 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 two additional

Re: RFR: 8349910: Implement JEP 517: HTTP/3 for the HTTP Client API [v7]

2025-06-06 Thread Daniel Fuchs
> Hi, > > Please find here a PR for the implementation of [JEP 517: HTTP/3 for the HTTP > Client API](https://openjdk.org/jeps/517). > > The CSR can be viewed at [JDK-8350588: Implement JEP 517: HTTP/3 for the HTTP > Client API](https://bugs.openjdk.org/browse/JDK-8350588) > > This JEP propose

Re: RFR: 8354242: VectorAPI: combine vector not operation with compare [v7]

2025-06-06 Thread erifan
On Fri, 6 Jun 2025 07:01:58 GMT, erifan wrote: > > > Oh I think we still cannot use `BoolTest::negate`, because we cannot > > > instantiate a `BoolTest` object with **unsigned** comparison. > > > `BoolTest::negate` is a non-static function. > > > > > > I see. Ok. Hmm. I still think that the l

Re: RFR: 8354242: VectorAPI: combine vector not operation with compare [v8]

2025-06-06 Thread erifan
> This patch optimizes the following patterns: > For integer types: > > (XorV (VectorMaskCmp src1 src2 cond) (Replicate -1)) > => (VectorMaskCmp src1 src2 ncond) > (XorVMask (VectorMaskCmp src1 src2 cond) (MaskAll m1)) > => (VectorMaskCmp src1 src2 ncond) > > cond can be eq, ne, le, ge, l

Re: RFR: 8358626: Emit UTF-8 CLDR resources [v2]

2025-06-06 Thread Erik Joelsson
On Thu, 5 Jun 2025 18:30:40 GMT, Naoto Sato wrote: >> Changes to generate CLDR resource bundles in UTF-8 encoding. The resource >> files in `java.base` are supposed to be US English only, but they also need >> to use UTF-8 as some of the names are non-ASCII (e.g., Türkiye) > > Naoto Sato has up

Re: RFR: 8077587: BigInteger Roots [v18]

2025-06-06 Thread fabioromano1
> This PR implements nth root computation for BigIntegers using Newton method. fabioromano1 has updated the pull request incrementally with one additional commit since the last revision: Update "since" version - Changes: - all: https://git.openjdk.org/jdk/pull/24898/files - n

Re: RFR: 8357708: com.sun.jndi.ldap.Connection ignores queued LDAP replies if Connection is subsequently closed

2025-06-06 Thread Jaikiran Pai
On Mon, 26 May 2025 16:52:56 GMT, Daniel Fuchs wrote: >> Hello Daniel, >> >>> I'd be tempted to say no - except that getReplyBer() will take the markers >>> out of the queue. >> >> That's correct - the change intentionally removes the lock and also lets the >> close/cancel markers land into t

Re: RFR: 8354242: VectorAPI: combine vector not operation with compare [v7]

2025-06-06 Thread erifan
On Thu, 5 Jun 2025 11:05:48 GMT, Emanuel Peter wrote: > > Oh I think we still cannot use `BoolTest::negate`, because we cannot > > instantiate a `BoolTest` object with **unsigned** comparison. > > `BoolTest::negate` is a non-static function. > > I see. Ok. Hmm. I still think that the logic sho

Re: RFR: 8358626: Emit UTF-8 CLDR resources [v2]

2025-06-06 Thread Volkan Yazici
On Thu, 5 Jun 2025 18:30:40 GMT, Naoto Sato wrote: >> Changes to generate CLDR resource bundles in UTF-8 encoding. The resource >> files in `java.base` are supposed to be US English only, but they also need >> to use UTF-8 as some of the names are non-ASCII (e.g., Türkiye) > > Naoto Sato has up