March 2021 - core-libs-dev

Re: RFR: 8264332: Use the blessed modifier order in jdk.charsets
by Alan Bateman 29 Mar '21

29 Mar '21

On Sun, 28 Mar 2021 13:56:00 GMT, Alex Blewitt <github.com+76791+alblue(a)openjdk.org> wrote: > 8264332: Use the blessed modifier order in jdk.charsets Marked as reviewed by alanb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/3236

1 0

RFR: 8264334: Use the blessed modifier order in jdk.jpackage
by Alex Blewitt 29 Mar '21

29 Mar '21

8264334: Use the blessed modifier order in jdk.jpackage ------------- Commit messages: - 8264334: Use the blessed modifier order in jdk.jpackage Changes: https://git.openjdk.java.net/jdk/pull/3234/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3234&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8264334 Stats: 58 lines in 19 files changed: 0 ins; 0 del; 58 mod Patch: https://git.openjdk.java.net/jdk/pull/3234.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3234/head:pull/3234 PR: https://git.openjdk.java.net/jdk/pull/3234

3 3

Dynamic Deserialization Filters - a JEP draft
by Roger Riggs 26 Mar '21

26 Mar '21

Please review a JEP to extend deserialization filtering (JEP 290) to enable deserialization filters to be selected by the application based on the runtime context. JDK-8263381 <https://bugs.openjdk.java.net/browse/JDK-8263381> Dynamic Deserialization Filters Comments appreciated, Roger

2 1

RFR: 8262110: DST starts from incorrect time in 2038
by Naoto Sato 26 Mar '21

26 Mar '21

Please review the fix to the DST transition bug after the year 2037. The logic had the side effect that it adjusted the dst offset every time the method `getOffsets()` is issued. Only adjust the offset when issued with wall time. ------------- Commit messages: - Set time zone to the formatter. - Added eof - Added a test case for 8073446 - 8262110: DST starts from incorrect time in 2038 Changes: https://git.openjdk.java.net/jdk/pull/3165/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3165&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8262110 Stats: 75 lines in 2 files changed: 73 ins; 0 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/3165.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3165/head:pull/3165 PR: https://git.openjdk.java.net/jdk/pull/3165

2 2

RFR: 8189198: Add "forRemoval = true" to Applet API deprecations
by Andy Herrick 26 Mar '21

26 Mar '21

implementation of JDK-8256145: JEP 398: Deprecate the Applet API for Removal ------------- Commit messages: - 8189198: Add "forRemoval = true" to Applet API deprecations Changes: https://git.openjdk.java.net/jdk/pull/2920/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2920&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8189198 Stats: 74 lines in 22 files changed: 14 ins; 0 del; 60 mod Patch: https://git.openjdk.java.net/jdk/pull/2920.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2920/head:pull/2920 PR: https://git.openjdk.java.net/jdk/pull/2920

6 15

RFR: 8263968: CDS: java/lang/ModuleLayer.EMPTY_LAYER should be singleton
by Aleksei Voitylov 26 Mar '21

26 Mar '21

With CDS and G1, test api/java_lang/ModuleLayer/Boot.html fails in JDK 16. After JDK-8253081 with CDS enabled two instances of empty layer appear in the runtime. One comes from default initialization of java/lang/ModuleLayer.EMPTY_LAYER, another one comes from CDS archive and is returned by ModuleLayer.boot().parents().get(0). The fix makes java/lang/ModuleLayer.EMPTY_LAYER a singleton with both CDS on and off, similar to java/lang/module/Configuration.EMPTY_CONFIGURATION. Testing: JCK 16, jtreg (including newly added test), pre-submit GitHub actions tests. ------------- Commit messages: - fix jcheck requirements - JDK-8263968: CDS: java/lang/ModuleLayer.EMPTY_LAYER should be singleton Changes: https://git.openjdk.java.net/jdk/pull/3131/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3131&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8263968 Stats: 24 lines in 3 files changed: 21 ins; 0 del; 3 mod Patch: https://git.openjdk.java.net/jdk/pull/3131.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3131/head:pull/3131 PR: https://git.openjdk.java.net/jdk/pull/3131

6 10

RFR: 8264161: BigDecimal#stripTrailingZeros can throw undocumented ArithmeticException
by Joe Darcy 25 Mar '21

25 Mar '21

While some of the existing text in BigDecimal could be read as implying exponent overflow/underflow will throw an ArithmeticException, it is reasonable to simply state that explicitly. The next text is meant to cover the usage of the checkScale family of methods. I'll reflow the existing paragraph once the text is agreed to. ------------- Commit messages: - 8264161: BigDecimal#stripTrailingZeros can throw undocumented ArithmeticException Changes: https://git.openjdk.java.net/jdk/pull/3204/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3204&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8264161 Stats: 7 lines in 1 file changed: 5 ins; 1 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/3204.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3204/head:pull/3204 PR: https://git.openjdk.java.net/jdk/pull/3204

2 4

RFR: 8253795: Implementation of JEP 391: macOS/AArch64 Port
by Anton Kozlov 25 Mar '21

25 Mar '21

Please review the implementation of JEP 391: macOS/AArch64 Port. It's heavily based on existing ports to linux/aarch64, macos/x86_64, and windows/aarch64. Major changes are in: * src/hotspot/cpu/aarch64: support of the new calling convention (subtasks JDK-8253817, JDK-8253818) * src/hotspot/os_cpu/bsd_aarch64: copy of os_cpu/linux_aarch64 with necessary adjustments (JDK-8253819) * src/hotspot/share, test/hotspot/gtest: support of write-xor-execute (W^X), required on macOS/AArch64 platform. It's implemented with pthread_jit_write_protect_np provided by Apple. The W^X mode is local to a thread, so W^X mode change relates to the java thread state change (for java threads). In most cases, JVM executes in write-only mode, except when calling a generated stub like SafeFetch, which requires a temporary switch to execute-only mode. The same execute-only mode is enabled when a java thread executes in java or native states. This approach of managing W^X mode turned out to be simple and efficient enough. * src/jdk.hotspot.agent: serviceability agent implementation (JDK-8254941) ------------- Commit messages: - Fix build - JDK-8253816: Update after recent changes - Rework gtests to always have wx_write - Revert gtest changes - Fix gtests in debug - Merge remote-tracking branch 'upstream/master' into jdk-macos - Fix windows_aarch64 - Use r18_tls instead of r18_reserved - JDK-8253742: bsd_aarch64 part - JDK-8257882: bsd_aarch64 part - ... and 40 more: https://git.openjdk.java.net/jdk/compare/82adfb32...3d4f4c23 Changes: https://git.openjdk.java.net/jdk/pull/2200/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2200&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8253795 Stats: 3335 lines in 117 files changed: 3230 ins; 41 del; 64 mod Patch: https://git.openjdk.java.net/jdk/pull/2200.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2200/head:pull/2200 PR: https://git.openjdk.java.net/jdk/pull/2200

26 231

RFR: 8264165: jpackage BasicTest fails after JDK-8220266: Check help text contains plaform specific parameters
by Alexey Semenyuk 25 Mar '21

25 Mar '21

Add missing escape single quote (') and typo fix ------------- Commit messages: - 8264165: jpackage BasicTest fails after JDK-8220266: Check help text contains plaform specific parameters Changes: https://git.openjdk.java.net/jdk/pull/3193/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3193&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8264165 Stats: 4 lines in 4 files changed: 0 ins; 0 del; 4 mod Patch: https://git.openjdk.java.net/jdk/pull/3193.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3193/head:pull/3193 PR: https://git.openjdk.java.net/jdk/pull/3193

3 3

RFR: 8220266: add support for additional metadata in add/remove programs
by Alexey Semenyuk 24 Mar '21

24 Mar '21

Add support for --about-url, --win-update-url and --win-help-url parameters ------------- Commit messages: - Remove unused import-s - 8220266: add support for additional metadata in add/remove programs Changes: https://git.openjdk.java.net/jdk/pull/3178/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3178&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8220266 Stats: 573 lines in 9 files changed: 563 ins; 0 del; 10 mod Patch: https://git.openjdk.java.net/jdk/pull/3178.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3178/head:pull/3178 PR: https://git.openjdk.java.net/jdk/pull/3178

3 3