August 2021 - core-libs-dev

Re: Issue with BuiltinClassLoader.ucp field not being visible
by Luke Hutchison 10 Aug '21

10 Aug '21

On Tue, Aug 10, 2021 at 12:51 AM Luke Hutchison <luke.hutch(a)gmail.com> wrote: > Could a getURLClassPath() method please be added to BuiltInClassLoader? > For security reasons, the getURLClassPath() method should return a copy of ucp, not the reference to ucp, to prevent the caller from modifying the system classpath.

2 3

RFR: 8264792: The NumberFormat for locale sq_XK formats price incorrectly.
by Naoto Sato 09 Aug '21

09 Aug '21

Please review the fix to the subject issue. The root cause of this problem is that the currency for the country code `XK` is undefined because the country code is user-defined in the ISO 3166 standard. However, it is commonly used to represent the region `Kosovo`, which CLDR supports and subsequently is supported by the JDK since JDK9. Adding the data `EUR` for the country code `XK` will fix the issue. ------------- Commit messages: - 8264792: The NumberFormat for locale sq_XK formats price incorrectly. Changes: https://git.openjdk.java.net/jdk/pull/5033/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5033&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8264792 Stats: 13 lines in 3 files changed: 6 ins; 0 del; 7 mod Patch: https://git.openjdk.java.net/jdk/pull/5033.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5033/head:pull/5033 PR: https://git.openjdk.java.net/jdk/pull/5033

3 3

Integrated: 8272095: ProblemList java/nio/channels/FileChannel/Transfer2GPlus.java on linux-aarch64
by Daniel D.Daugherty 06 Aug '21

06 Aug '21

A trivial fix to ProblemList java/nio/channels/FileChannel/Transfer2GPlus.java on linux-aarch64 ------------- Commit messages: - 8272095: ProblemList java/nio/channels/FileChannel/Transfer2GPlus.java on linux-aarch64 Changes: https://git.openjdk.java.net/jdk/pull/5032/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5032&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8272095 Stats: 2 lines in 1 file changed: 2 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/5032.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5032/head:pull/5032 PR: https://git.openjdk.java.net/jdk/pull/5032

2 3

RFR: 8271868: Warn user when using mac-sign option with unsigned app-image.
by Andy Herrick 06 Aug '21

06 Aug '21

8271868: Warn user when using mac-sign option with unsigned app-image. ------------- Commit messages: - JDK-8271868: Warn user when using mac-sign option with unsigned app-image. - JDK-8271868: Warn user when using mac-sign option with unsigned app-image. - JDK-8271868: Warn user when using mac-sign option with unsigned app-image. - JDK-8271868: Warn user when using mac-sign option with unsigned app-image. Changes: https://git.openjdk.java.net/jdk/pull/5004/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5004&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8271868 Stats: 34 lines in 5 files changed: 32 ins; 0 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/5004.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5004/head:pull/5004 PR: https://git.openjdk.java.net/jdk/pull/5004

4 15

RFR: 8271208: Typo in ModuleDescriptor.read javadoc
by Jaikiran Pai 06 Aug '21

06 Aug '21

Can I please get a review of this trivial fix which fixes the typo noted in https://bugs.openjdk.java.net/browse/JDK-8271208? Ran `make docs-image` locally and generated the new javadocs and the change looks fine. ------------- Commit messages: - 8271208: Typo in ModuleDescriptor.read javadoc Changes: https://git.openjdk.java.net/jdk/pull/5021/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5021&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8271208 Stats: 3 lines in 1 file changed: 0 ins; 0 del; 3 mod Patch: https://git.openjdk.java.net/jdk/pull/5021.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5021/head:pull/5021 PR: https://git.openjdk.java.net/jdk/pull/5021

3 4

RFR: Merge jdk17
by Jesper Wilhelmsson 06 Aug '21

06 Aug '21

Forwardport JDK 17 -> JDK 18 ------------- Commit messages: - Merge - 8270872: Final nroff manpage update for JDK 17 - 8271588: JFR Recorder Thread crashed with SIGSEGV in write_klass - 8271863: ProblemList serviceability/sa/TestJmapCore.java on linux-x64 with ZGC The webrevs contain the adjustments done while merging with regards to each parent branch: - master: https://webrevs.openjdk.java.net/?repo=jdk&pr=5026&range=00.0 - jdk17: https://webrevs.openjdk.java.net/?repo=jdk&pr=5026&range=00.1 Changes: https://git.openjdk.java.net/jdk/pull/5026/files Stats: 307 lines in 32 files changed: 129 ins; 33 del; 145 mod Patch: https://git.openjdk.java.net/jdk/pull/5026.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5026/head:pull/5026 PR: https://git.openjdk.java.net/jdk/pull/5026

1 2

[jdk17] RFR: JDK-8270872: Final nroff manpage update for JDK 17
by Jonathan Gibbons 05 Aug '21

05 Aug '21

Please review a semi-automatic update of the nroff man pages from the upstream files. ------------- Commit messages: - JDK-8270872: Final nroff manpage update for JDK 17 Changes: https://git.openjdk.java.net/jdk17/pull/303/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk17&pr=303&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8270872 Stats: 289 lines in 27 files changed: 117 ins; 31 del; 141 mod Patch: https://git.openjdk.java.net/jdk17/pull/303.diff Fetch: git fetch https://git.openjdk.java.net/jdk17 pull/303/head:pull/303 PR: https://git.openjdk.java.net/jdk17/pull/303

5 9

New candidate JEP: 416: Reimplement Core Reflection with Method Handles
by mark.reinhold＠oracle.com 05 Aug '21

05 Aug '21

https://openjdk.java.net/jeps/416 Summary: Reimplement java.lang.reflect.Method, Constructor, and Field on top of java.lang.invoke method handles. Making method handles the underlying mechanism for reflection will reduce the maintenance and development cost of both the java.lang.reflect and java.lang.invoke APIs. - Mark

1 0

RFR: 8271840: Add simple Integer.toString microbenchmarks
by Claes Redestad 05 Aug '21

05 Aug '21

This adds Integer.toString microbenchmarks, roughly similar to the Long.toString microbenchmarks (toStringSmall should use an equivalent data set). This is useful for comparison purposes and completeness. Also tuned existing benchmarks to harmonize setup, reduce runtime without significantly harming reliability of results, and remove the explicit Threads.MAX from Longs ------------- Commit messages: - Add simple Integer.toString microbenchmarks Changes: https://git.openjdk.java.net/jdk/pull/4988/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4988&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8271840 Stats: 43 lines in 2 files changed: 33 ins; 3 del; 7 mod Patch: https://git.openjdk.java.net/jdk/pull/4988.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/4988/head:pull/4988 PR: https://git.openjdk.java.net/jdk/pull/4988

2 3

Integrated: 8271888: build error after JDK-8271599
by Joe Darcy 04 Aug '21

04 Aug '21

Remove extraneous <p> tag to fix build breakage. ------------- Commit messages: - 8271888 Changes: https://git.openjdk.java.net/jdk/pull/5002/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5002&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8271888 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/5002.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5002/head:pull/5002 PR: https://git.openjdk.java.net/jdk/pull/5002

3 3