Re: RFR: 8367619: String.format in outOfRangeException uses wrong format specifier for String argument

2025-09-17 Thread Francesco Andreuzzi
On Mon, 15 Sep 2025 05:37:58 GMT, Guanqiang Han wrote: > trivial fix Marked as reviewed by fandreuzzi (Author). - PR Review: https://git.openjdk.org/jdk/pull/27280#pullrequestreview-3223443676

Re: RFR: 8355342: File.getCanonicalPath on Java 24 resolves paths on network drives to UNC format [v2]

2025-09-17 Thread Francesco Andreuzzi
On Tue, 16 Sep 2025 22:03:54 GMT, Brian Burkhalter wrote: >> `File.getCanonicalPath` invokes `GetFinalPathNameByHandle` on the result of >> `canonicalize0` which causes the drive letter of a mapped drive to be >> converted to a UNC prefix. If such a substitution is detected, this request >> pr

Integrated: 8354871: Replace stack map frame type magics with constants

2025-09-07 Thread Francesco Andreuzzi
On Mon, 1 Sep 2025 09:00:32 GMT, Francesco Andreuzzi wrote: > In this PR I introduce some constants in `StackMapGenerator`. No change in > logic is expected. > > Passes tests in `jdk/classfile`. This pull request has now been integrated. Changeset: 8a6b8751 Author: Frances

Re: RFR: 8354871: Replace stack map frame type magics with constants [v3]

2025-09-06 Thread Francesco Andreuzzi
On Sat, 6 Sep 2025 13:55:03 GMT, Chen Liang wrote: >> Francesco Andreuzzi has updated the pull request incrementally with one >> additional commit since the last revision: >> >> inclusive > > Reviewer's note: This brings more parity with c++ veri

Re: RFR: 8354871: Replace stack map frame type magics with constants [v3]

2025-09-06 Thread Francesco Andreuzzi
On Sat, 6 Sep 2025 13:22:56 GMT, Chen Liang wrote: >> Francesco Andreuzzi has updated the pull request incrementally with one >> additional commit since the last revision: >> >> inclusive > > src/java.base/share/classes/jdk/internal/classfile/impl/Stac

Re: RFR: 8354871: Replace stack map frame type magics with constants [v3]

2025-09-06 Thread Francesco Andreuzzi
> In this PR I introduce some constants in `StackMapGenerator`. No change in > logic is expected. > > Passes tests in `jdk/classfile`. Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision: inclusive --

Re: RFR: 8354871: Replace stack map frame type magics with constants [v2]

2025-09-06 Thread Francesco Andreuzzi
> In this PR I introduce some constants in `StackMapGenerator`. No change in > logic is expected. > > Passes tests in `jdk/classfile`. Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision: Apply suggestions from

Re: RFR: 8354871: Replace stack map frame type magics with constants

2025-09-06 Thread Francesco Andreuzzi
On Mon, 1 Sep 2025 09:00:32 GMT, Francesco Andreuzzi wrote: > In this PR I introduce some constants in `StackMapGenerator`. No change in > logic is expected. > > Passes tests in `jdk/classfile`. Hi @liach, could you have a look at this PR since you opened the ticket? -

RFR: 8354871: Replace stack map frame type magics with constants

2025-09-01 Thread Francesco Andreuzzi
In this PR I introduce some constants in `StackMapGenerator`. No change in logic is expected. Passes tests in `jdk/classfile`. - Commit messages: - nn - Merge branch 'master' into JDK-8354871 - constants Changes: https://git.openjdk.org/jdk/pull/27029/files Webrev: https://web

Integrated: 8361593: Commented dead code in JDK-8342868 can be removed

2025-08-30 Thread Francesco Andreuzzi
On Fri, 29 Aug 2025 20:12:42 GMT, Francesco Andreuzzi wrote: > [JDK-8342868](https://bugs.openjdk.org/browse/JDK-8342868) commented out code > rather than removing it. The code can be removed safely, as pointed out in > the PR review: #21654. This pull request has now been i

RFR: 8361593: Commented dead code in JDK-8342868 can be removed

2025-08-29 Thread Francesco Andreuzzi
[JDK-8342868](https://bugs.openjdk.org/browse/JDK-8342868) commented out code rather than removing it. The code can be removed safely, as pointed out in the PR review: #21654. - Commit messages: - remove commented code Changes: https://git.openjdk.org/jdk/pull/27012/files Webrev

Re: RFR: 8366102: Clarification Needed: Symbolic Link Handling in File API Specifications

2025-08-29 Thread Francesco Andreuzzi
On Fri, 29 Aug 2025 15:13:52 GMT, Brian Burkhalter wrote: >> src/java.base/share/classes/java/io/File.java line 85: >> >>> 83: * Unless otherwise noted, {@linkplain java.nio.file##links >>> symbolic links} >>> 84: * are automatically redirected to the target of the link, >>> whether they >>

Re: RFR: 8366102: Clarification Needed: Symbolic Link Handling in File API Specifications

2025-08-29 Thread Francesco Andreuzzi
On Fri, 29 Aug 2025 15:01:51 GMT, Brian Burkhalter wrote: > Attempt to improve the verbiage of the `java.io.File` specification to make > it clearer whether a given method operates only on an abstract pathname or on > the actual file system entry it denotes. src/java.base/share/classes/java/io

Re: RFR: 8366261: Provide utility methods for sun.security.util.Password [v2]

2025-08-28 Thread Francesco Andreuzzi
On Wed, 27 Aug 2025 23:48:28 GMT, Naoto Sato wrote: >> Providing a couple of utility methods using the "built-in" `Console` >> implementation to support tools that require password input, such as >> `keytool`, ensuring they work even when stdin is redirected. > > Naoto Sato has updated the pull

Re: [jdk25] RFR: 8361497: Scoped Values: orElse and orElseThrow do not access the cache

2025-08-28 Thread Francesco Andreuzzi
On Thu, 28 Aug 2025 01:14:20 GMT, Rui Li wrote: > Backport of https://bugs.openjdk.org/browse/JDK-8361497 > > Not clean due to the lack of https://bugs.openjdk.org/browse/JDK-8360884. The > cherry pick conflict was caused by the a new test added in the JDK-8360884., > [code](https://github.com

Re: RFR: 8356218: [macos] Document --app-content [v2]

2025-08-19 Thread Francesco Andreuzzi
On Tue, 19 Aug 2025 22:10:53 GMT, Alexander Matveev wrote: >> - Added following note for `--app-content` on macOS to help and man page: >> `The value should be a directory with the "Resources" subdirectory (or any >> other directory that is valid in the "Contents" directory of the application

Re: RFR: 8364822: Comment cleanup, stale references to closeDescriptors and UNIXProcess.c

2025-08-07 Thread Francesco Andreuzzi
On Thu, 7 Aug 2025 01:49:41 GMT, Guanqiang Han wrote: > Clean up some stale comment references. Trivial changes. src/hotspot/os/linux/os_linux.cpp line 4876: > 4874: // flag set. If we don't set it, then careless 3rd party native code > 4875: // might fork and exec without closing all app

Re: RFR: 8364361: [process] java.lang.Process should implement close and be AutoCloseable

2025-08-05 Thread Francesco Andreuzzi
On Tue, 5 Aug 2025 18:21:24 GMT, Roger Riggs wrote: > The teardown of a Process launched by `ProcessBuilder` includes the closing > of streams and ensuring the termination of the process is the responsibility > of the caller. The `Process.close()` method provides a clear and obvious way > to e

Re: RFR: 8363972: Loose matching of dash/minusSign in number parsing [v4]

2025-08-01 Thread Francesco Andreuzzi
On Thu, 31 Jul 2025 22:08:50 GMT, Naoto Sato wrote: >> src/java.base/share/classes/java/text/DecimalFormatSymbols.java line 1002: >> >>> 1000: >>> 1001: if (loadNumberData(locale) instanceof Object[] d && >>> 1002: d[0] instanceof String[] numberElements) { >> >> Should the

Re: RFR: 8363972: Loose matching of dash/minusSign in number parsing

2025-07-31 Thread Francesco Andreuzzi
On Thu, 31 Jul 2025 18:41:47 GMT, Naoto Sato wrote: > Enabling lenient minus sign matching when parsing numbers. In some locales, > e.g. Finnish, the default minus sign is the Unicode "Minus Sign" (U+2212), > which is not the "Hyphen Minus" (U+002D) that users type in from keyboard. > Thus the