November 2021 - core-libs-dev

RFR: 8276048: Error in javadoc regarding Long method public static Long valueOf(String s)
by swati sharma 03 Dec '21

03 Dec '21

8276048: Error in javadoc regarding Long method public static Long valueOf(String s) Fix: Changing integer to {@code Long} ------------- Commit messages: - Update Long.java Changes: https://git.openjdk.java.net/jdk/pull/6135/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6135&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8276048 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/6135.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6135/head:pull/6135 PR: https://git.openjdk.java.net/jdk/pull/6135

4 4

RFR: JDK-8276674 : Malformed Javadoc inline tags in JDK source in java/util/stream
by Tim Prinzing 02 Dec '21

02 Dec '21

JDK-8276674 : Malformed Javadoc inline tags in JDK source in java/util/stream ------------- Commit messages: - Fixed @code and @link in some javadoc for JDK-8276674 Changes: https://git.openjdk.java.net/jdk/pull/6443/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6443&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8276674 Stats: 14 lines in 9 files changed: 0 ins; 0 del; 14 mod Patch: https://git.openjdk.java.net/jdk/pull/6443.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6443/head:pull/6443 PR: https://git.openjdk.java.net/jdk/pull/6443

5 9

RFR: 8276806: Use Objects.checkFromIndexSize where possible in java.base
by Сергей Цыпанов 02 Dec '21

02 Dec '21

This is a follow-up for https://github.com/openjdk/jdk/pull/4507, looks like there are some cases that were not covered. Also this is related to https://github.com/openjdk/jdk/pull/3615 ------------- Commit messages: - 8276806: Use Objects.checkFromIndexSize where possible in java.base Changes: https://git.openjdk.java.net/jdk/pull/6297/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6297&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8276806 Stats: 55 lines in 11 files changed: 11 ins; 13 del; 31 mod Patch: https://git.openjdk.java.net/jdk/pull/6297.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6297/head:pull/6297 PR: https://git.openjdk.java.net/jdk/pull/6297

4 6

RFR: 8276904: Optional.toString() is unnecessarily expensive
by Eamonn McManus 02 Dec '21

02 Dec '21

Use string concatenation instead of `String.format`. ------------- Commit messages: - 8276904: Optional.toString() is unnecessarily expensive Changes: https://git.openjdk.java.net/jdk/pull/6337/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6337&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8276904 Stats: 4 lines in 4 files changed: 0 ins; 0 del; 4 mod Patch: https://git.openjdk.java.net/jdk/pull/6337.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6337/head:pull/6337 PR: https://git.openjdk.java.net/jdk/pull/6337

4 3

RFR: 8275342: Change nested classes in java.prefs to static nested classes
by Andrey Turbanov 02 Dec '21

02 Dec '21

Non-static classes hold a link to their parent classes, which can be avoided. ------------- Commit messages: - [PATCH] Change nested classes in java.prefs to static nested classes Changes: https://git.openjdk.java.net/jdk/pull/5971/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5971&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8275342 Stats: 20 lines in 3 files changed: 1 ins; 11 del; 8 mod Patch: https://git.openjdk.java.net/jdk/pull/5971.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5971/head:pull/5971 PR: https://git.openjdk.java.net/jdk/pull/5971

3 4

RFR: 8277322: Document that setting an invalid property `jdk.serialFilter` disables deserialization
by Roger Riggs 02 Dec '21

02 Dec '21

The effects of an invalid `jdk.serialFilter` property are not completely documented. If the value of the system property jdk.serialFilter is invalid, deserialization should not be possible and it should be clear in the specification. Specify an implementation specific exception is thrown in the case where deserialization is invoked after reporting the invalid jdk.serialFilter. ------------- Commit messages: - 8277322: Document that setting an invalid property `jdk.serialFilter` disables deserialization Changes: https://git.openjdk.java.net/jdk/pull/6508/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6508&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8277322 Stats: 5 lines in 1 file changed: 2 ins; 0 del; 3 mod Patch: https://git.openjdk.java.net/jdk/pull/6508.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6508/head:pull/6508 PR: https://git.openjdk.java.net/jdk/pull/6508

4 10

RFR: JDK-8273056 java.util.random does not correctly sample exponential or Gaussian distributions
by Jim Laskey 02 Dec '21

02 Dec '21

The modified ziggurat algorithm is not correctly implemented in `java.base/jdk/internal/util/random/RandomSupport.java`. Create a histogram of a million samples using 2000 uniform bins with the following range: Exponential range from 0 to 12. Gaussian range from -8 to 8. This does not pass a Chi-square test. If you look at the histogram it is obviously not showing the shape of the PDF for these distributions. Look closely at the range around zero (e.g. +/- 0.5). ------------- Commit messages: - 8273056 - java.util.random does not correctly sample exponential or Gaussian distributions Changes: https://git.openjdk.java.net/jdk/pull/6353/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6353&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8273056 Stats: 15 lines in 1 file changed: 6 ins; 4 del; 5 mod Patch: https://git.openjdk.java.net/jdk/pull/6353.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6353/head:pull/6353 PR: https://git.openjdk.java.net/jdk/pull/6353

3 9

RFR: 8277924: Small tweaks to foreign function and memory API
by Maurizio Cimadamore 02 Dec '21

02 Dec '21

Following integration of the second incubator of the foreign function and memory API [1], we detected few divergences between the contents of the jdk repo and the panama repo: * the name of some of the `FunctionDescriptor` wither methods is different (e.g. `withAppendedLayoutArguments` vs. `appendLayoutArguments`), as it has been simplified and improved following a change that was not incorporated in [1]. * TestUpcall does not execute all the test combinations, because of an issue in the jtreg header (also fixed in the panama repo) * Addressing some feedback, we would like to bring back alignment to JAVA_INT layout constants (and related constants). Javadoc: http://cr.openjdk.java.net/~mcimadamore/8277924/v1/javadoc/jdk/incubator/fo… Specdiff: http://cr.openjdk.java.net/~mcimadamore/8277924/v1/spec_diff/overview-summa… [1] - #5907 ------------- Commit messages: - Further tweak to javadoc of layout constants - Wrong alignment constraints in ValueLayout constants javadoc - Tweak javadoc - Fix jtreg header on TestUpcall - Tweak API names for FunctionDescriptor methods - * Tweak value layout constants alignment to reflect VM alignment Changes: https://git.openjdk.java.net/jdk/pull/6589/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6589&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8277924 Stats: 343 lines in 14 files changed: 206 ins; 5 del; 132 mod Patch: https://git.openjdk.java.net/jdk/pull/6589.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6589/head:pull/6589 PR: https://git.openjdk.java.net/jdk/pull/6589

3 11

RFR: 8276657: XSLT compilter tries to define a class with empty name
by Joe Wang 02 Dec '21

02 Dec '21

The result of Util.baseName(systemId) can be empty, causing the compiler to set an empty classname. Add a check to make sure it will not set the empty classname. Alternatively, it may report an error, but that would be disruptive. As the transform can proceed without the provided classname (by using the default), adding a check is better than reporting an error. I've verified the patch with the proposed fix for JDK-8276241. Harold has also confirmed it fixes the tests in his builds. ------------- Commit messages: - 8276657: XSLT compilter tries to define a class with empty name Changes: https://git.openjdk.java.net/jdk/pull/6620/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6620&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8276657 Stats: 5 lines in 1 file changed: 3 ins; 0 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/6620.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6620/head:pull/6620 PR: https://git.openjdk.java.net/jdk/pull/6620

2 4

RFR: 8276141: XPathFactory set/getProperty method
by Joe Wang 02 Dec '21

02 Dec '21

Add setProperty/getProperty methods to the XPath API so that properties can be supported in the future. ------------- Commit messages: - 8276141: XPathFactory set/getProperty method Changes: https://git.openjdk.java.net/jdk/pull/6215/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6215&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8276141 Stats: 51 lines in 1 file changed: 51 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/6215.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/6215/head:pull/6215 PR: https://git.openjdk.java.net/jdk/pull/6215

6 35