February 2021 - core-libs-dev

RFR: 8262042: ProblemList javax/xml/jaxp/unittest/common/prettyprint/PrettyPrintTest.java on Windows
by Daniel D.Daugherty 19 Feb '21

19 Feb '21

A trivial fix to ProblemList javax/xml/jaxp/unittest/common/prettyprint/PrettyPrintTest.java on Windows. ------------- Commit messages: - 8262042: ProblemList javax/xml/jaxp/unittest/common/prettyprint/PrettyPrintTest.java on Windows Changes: https://git.openjdk.java.net/jdk/pull/2650/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2650&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8262042 Stats: 3 lines in 1 file changed: 2 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/2650.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2650/head:pull/2650 PR: https://git.openjdk.java.net/jdk/pull/2650

2 4

RFR: 8247918: Clarify Reader.skip behavior for end of stream
by Brian Burkhalter 19 Feb '21

19 Feb '21

Please review this clarification of the specification of the method `skip(long)` in `java.io.Reader` and its subclasses. Specifically, the behavior of the method is made clear for the case when the `Reader` is already at the end of its stream when the method is invoked. A corresponding CSR will be filed. Also, the change includes an update to an existing test in order to verify that the specification change reflects actual behavior. ------------- Commit messages: - 8247918: Clarify Reader.skip behavior for end of stream Changes: https://git.openjdk.java.net/jdk/pull/2274/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2274&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8247918 Stats: 132 lines in 8 files changed: 62 ins; 38 del; 32 mod Patch: https://git.openjdk.java.net/jdk/pull/2274.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2274/head:pull/2274 PR: https://git.openjdk.java.net/jdk/pull/2274

5 14

RFR: 8261728: SimpleDateFormat should link to DateTimeFormatter
by Naoto Sato 19 Feb '21

19 Feb '21

Please review this simple doc fix. A CSR will be filed accordingly. ------------- Commit messages: - 8261728: SimpleDateFormat should link to DateTimeFormatter Changes: https://git.openjdk.java.net/jdk/pull/2616/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2616&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8261728 Stats: 8 lines in 2 files changed: 4 ins; 0 del; 4 mod Patch: https://git.openjdk.java.net/jdk/pull/2616.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2616/head:pull/2616 PR: https://git.openjdk.java.net/jdk/pull/2616

5 10

RFR: 6245663: (spec) File.renameTo(File) changes the file-system object, not the File instance
by Brian Burkhalter 19 Feb '21

19 Feb '21

Please review this minor specification update to highlight that `File.renameTo(File)` does not modify the `File` instance on which the method is invoked. ------------- Commit messages: - 6245663: (spec) File.renameTo(File) changes the file-system object, not the File instance Changes: https://git.openjdk.java.net/jdk/pull/2618/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2618&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-6245663 Stats: 4 lines in 1 file changed: 3 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/2618.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2618/head:pull/2618 PR: https://git.openjdk.java.net/jdk/pull/2618

4 16

RFR: 8261744: Implement CharsetDecoder ASCII and latin-1 fast-paths
by Claes Redestad 19 Feb '21

19 Feb '21

This patch exposes a couple of intrinsics used by String to speed up ASCII checking and byte[] -> char[] inflation, which can be used by latin1 and ASCII-compatible CharsetDecoders to speed up decoding operations. - Fast-path implemented for all standard charsets, with up to 10x performance improvements in microbenchmarks reading Strings from ByteArrayInputStream. - Cleanup of StreamDecoder/-Encoder with some minor improvements when interpreting - Reworked creation of JavaLangAccess to be safely published for CharsetDecoders/-Encoders used for setting up System.out/in. As JLA and these encoders are created during System.initPhase1 the current sequence caused the initialization to became unstable and a few tests were consistently getting an NPE. Testing: tier1-3 ------------- Commit messages: - Safer publication of JavaLangAccess - Logic error - Move creation of JavaLangAccess up before creating System.in/out - Fix typo (caught by Test4625418.java) - Merge branch 'master' into streamdecoder - Let decoders use String intrinsics to inflate ASCII-only bytes - Remove trailing spaces - Add microbenchmark - Simplify and improve StreamDecoder/-Encoder Changes: https://git.openjdk.java.net/jdk/pull/2574/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2574&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8261744 Stats: 706 lines in 13 files changed: 300 ins; 107 del; 299 mod Patch: https://git.openjdk.java.net/jdk/pull/2574.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2574/head:pull/2574 PR: https://git.openjdk.java.net/jdk/pull/2574

4 21

RFR: 8262018: Wrong format in SAP copyright header of OsVersionTest
by Christoph Langer 19 Feb '21

19 Feb '21

The fix for JDK-8261753 introduced a ',' after the copyright years which is not the expected format for SAP copyrights. ------------- Commit messages: - JDK-8262018 Changes: https://git.openjdk.java.net/jdk/pull/2642/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2642&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8262018 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/2642.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2642/head:pull/2642 PR: https://git.openjdk.java.net/jdk/pull/2642

2 2

RFR: JDK-8260858: Implementation specific property xsltcIsStandalone for XSLTC Serializer
by Joe Wang 19 Feb '21

19 Feb '21

Adds a property similar to 'isStandalone' in JDK-8249867. Please note that the test received an auto-format. The material changes were the two tests marked with bug id 8260858 and related data and methods. ------------- Commit messages: - JDK-8260858: Implementation specific property xsltcIsStandalone for XSLTC Serializer Changes: https://git.openjdk.java.net/jdk/pull/2559/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2559&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8260858 Stats: 243 lines in 7 files changed: 167 ins; 4 del; 72 mod Patch: https://git.openjdk.java.net/jdk/pull/2559.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2559/head:pull/2559 PR: https://git.openjdk.java.net/jdk/pull/2559

3 6

System.getEnv(String name, String def)
by Loïc MATHIEU 19 Feb '21

19 Feb '21

Hello, I wonder if there has already been some discussion to provide a System.getEnv(String name, String def) method that allows to return a default value in case the env variable didn't exist. When using system properties instead of env variable, we do have a System.getProperty(String key, String def) variant. Stating the JavaDoc of System.getEnv(): *System properties and environment variables are both conceptually mappings between names and values* So if system properties and environment variables are similar concepts, they should provide the same functionalities right ? This would be very convenient as more and more people rely on environment variables these days to configure their applications. Regards, Loïc

5 6

RFR: 8261977: Fix comment for getPrefixed() in canonicalize_md.c
by Alexey Semenyuk 18 Feb '21

18 Feb '21

Follow up for JDK-8235397 fix ------------- Commit messages: - Fix comment for getPrefixed() in canonicalize_md.c Changes: https://git.openjdk.java.net/jdk/pull/2631/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2631&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8261977 Stats: 5 lines in 1 file changed: 0 ins; 4 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/2631.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2631/head:pull/2631 PR: https://git.openjdk.java.net/jdk/pull/2631

2 2

RFR: 8261940: Fix references to IOException in BigDecimal javadoc
by Joe Darcy 18 Feb '21

18 Feb '21

Noticed by some of Jon Gibbons's doc linting & checking tooling, this changeset fixes two javadoc issues for BigDecimal's serialization-related methods, improving the serial form page. ------------- Commit messages: - 8261940: Fix references to IOException in BigDecimal javadoc Changes: https://git.openjdk.java.net/jdk/pull/2623/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2623&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8261940 Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/2623.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2623/head:pull/2623 PR: https://git.openjdk.java.net/jdk/pull/2623

5 7