Re: RFR: 8290316: Ensure that all directory streams are closed in java.base [v2]

2022-07-15 Thread Bernd
On Fri, 15 Jul 2022 19:33:56 GMT, Ryan Ernst wrote: >> This commit guards uses of Files methods returning path streams in >> java.base to use try-with-resources. > > Ryan Ernst has updated the pull request incrementally with one additional > commit since the last revision: > > address

Re: RFR: JDK-8289551: Conversions between bit representations of half precision values and floats [v3]

2022-07-15 Thread Joe Darcy
> Initial implementation. Joe Darcy has updated the pull request incrementally with one additional commit since the last revision: Preserve NaN sign bit. - Changes: - all: https://git.openjdk.org/jdk/pull/9422/files - new:

Re: RFR: JDK-8289551: Conversions between bit representations of half precision values and floats [v2]

2022-07-15 Thread Joe Darcy
> Initial implementation. Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 15 additional commits since the last revision: - Respond to

Re: RFR: JDK-8289551: Conversions between bit representations of half precision values and floats

2022-07-15 Thread Joe Darcy
On Wed, 13 Jul 2022 13:11:33 GMT, Raffaello Giulietti wrote: >> Initial implementation. > > src/java.base/share/classes/java/lang/Float.java line 1174: > >> 1172: >> 1173: short result = 0; >> 1174: result = (short)(((exp + 15) << 10) | signif_bits); > > result =

Re: RFR: 8290316: Ensure that all directory streams are closed in java.base

2022-07-15 Thread Ryan Ernst
On Fri, 15 Jul 2022 16:06:21 GMT, Ryan Ernst wrote: > This commit guards uses of Files methods returning path streams in > java.base to use try-with-resources. Thanks Alan. The 8 space indents were unintentional. I think I've addressed all your comments in

Re: RFR: 8290316: Ensure that all directory streams are closed in java.base [v2]

2022-07-15 Thread Ryan Ernst
> This commit guards uses of Files methods returning path streams in > java.base to use try-with-resources. Ryan Ernst has updated the pull request incrementally with one additional commit since the last revision: address formatting feedback - Changes: - all:

Integrated: 8290327: Remove java/lang/reflect/callerCache/ReflectionCallerCacheTest.java from ProblemList-Xcomp.txt

2022-07-15 Thread Mandy Chung
On Thu, 14 Jul 2022 21:01:05 GMT, Mandy Chung wrote: > It's expected that [JDK-8287596](https://bugs.openjdk.org/browse/JDK-8287596) > has resolved [JDK-8288286](https://bugs.openjdk.org/browse/JDK-8288286). > Remove ReflectionCallerCacheTest.java from the problem list for test > execution.

Re: RFR: 8290327: Remove java/lang/reflect/callerCache/ReflectionCallerCacheTest.java from ProblemList-Xcomp.txt

2022-07-15 Thread Mandy Chung
On Thu, 14 Jul 2022 21:01:05 GMT, Mandy Chung wrote: > It's expected that [JDK-8287596](https://bugs.openjdk.org/browse/JDK-8287596) > has resolved [JDK-8288286](https://bugs.openjdk.org/browse/JDK-8288286). > Remove ReflectionCallerCacheTest.java from the problem list for test > execution.

Re: RFR: 8289919: [test] LoadLibraryUnloadTest.java failed with "Failed to unload native library" [v4]

2022-07-15 Thread Mandy Chung
On Fri, 15 Jul 2022 15:31:45 GMT, Roger Riggs wrote: >> The test `java/lang/ClassLoader/loadLibraryUnload/LoadLibraryUnloadTest.java` >> Fails intermittently when expected output from a subprocess is not found. >> >> I suspect a race between the Cleaner that is going to call JNI_OnUnload (in

Re: RFR: 8290316: Ensure that all directory streams are closed in java.base

2022-07-15 Thread Alan Bateman
On Fri, 15 Jul 2022 16:06:21 GMT, Ryan Ernst wrote: > This commit guards uses of Files methods returning path streams in > java.base to use try-with-resources. src/java.base/share/classes/jdk/internal/module/ModuleHashes.java line 119: > 117: } > 118: try { > 119:

Re: RFR: 8290300: Use standard String-joining tools where applicable [v2]

2022-07-15 Thread Naoto Sato
On Fri, 15 Jul 2022 12:40:50 GMT, Сергей Цыпанов wrote: >> Simplify code with `String.join()` > > Сергей Цыпанов has updated the pull request incrementally with two additional > commits since the last revision: > > - 8290300: Revert jops > - 8290300: Revert jops Changes to `Locale` look

RFR: 8290359: Ensure that all directory streams are closed in jdk.link

2022-07-15 Thread Ryan Ernst
This commit adds try-with-resources for uses of Stream from Files methods that walk directories. - Commit messages: - 8290359: Ensure that all directory streams are closed in jdk.link Changes: https://git.openjdk.org/jdk/pull/9520/files Webrev:

RFR: 8290316: Ensure that all directory streams are closed in java.base

2022-07-15 Thread Ryan Ernst
This commit guards uses of Files methods returning path streams in java.base to use try-with-resources. - Commit messages: - 8290316: Ensure that all directory streams are closed in java.base Changes: https://git.openjdk.org/jdk/pull/9518/files Webrev:

Re: RFR: 8289919: [test] LoadLibraryUnloadTest.java failed with "Failed to unload native library" [v4]

2022-07-15 Thread Roger Riggs
> The test `java/lang/ClassLoader/loadLibraryUnload/LoadLibraryUnloadTest.java` > Fails intermittently when expected output from a subprocess is not found. > > I suspect a race between the Cleaner that is going to call JNI_OnUnload (in > NativeLibraries.java:377) when the ClassLoader is no

Re: Case Insensitive DateFormatSymbols for parsing

2022-07-15 Thread Bernd Eckenfels
Hello Roger, Thanks for the Tipp, that’s indeed a good reason why it’s not needed in the legacy SDF API. Gruss Bernd -- http://bernd.eckenfels.net Von: core-libs-dev im Auftrag von Roger Riggs Gesendet: Friday, July 15, 2022 4:14:16 PM An:

Re: Case Insensitive DateFormatSymbols for parsing

2022-07-15 Thread Roger Riggs
Hi Bernd, Perhaps use java.time.format.DateTimeFormatter. You can create a case-insensitive DateTimeFormatterBuilder using .parseCaseInsensitive(). https://docs.oracle.com/en/java/javase/18/docs/api/java.base/java/time/format/DateTimeFormatterBuilder.html#parseCaseInsensitive() Regards,

Re: RFR: 8290300: Use standard String-joining tools where applicable [v2]

2022-07-15 Thread Сергей Цыпанов
On Fri, 15 Jul 2022 12:25:07 GMT, Jim Laskey wrote: >> Сергей Цыпанов has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - 8290300: Revert jops >> - 8290300: Revert jops > > LGTM, however are there tests that ensure the changes are

Re: RFR: 8290300: Use standard String-joining tools where applicable

2022-07-15 Thread Сергей Цыпанов
On Fri, 15 Jul 2022 12:03:13 GMT, Сергей Цыпанов wrote: > Simplify code with `String.join()` Reverted jops - PR: https://git.openjdk.org/jdk/pull/9513

Re: RFR: 8290300: Use standard String-joining tools where applicable

2022-07-15 Thread Alan Bateman
On Fri, 15 Jul 2022 12:03:13 GMT, Сергей Цыпанов wrote: > Simplify code with `String.join()` joptsimple is a 3rd party code so we probably don't want to change that. - PR: https://git.openjdk.org/jdk/pull/9513

Re: RFR: 8290300: Use standard String-joining tools where applicable

2022-07-15 Thread Jim Laskey
On Fri, 15 Jul 2022 12:03:13 GMT, Сергей Цыпанов wrote: > Simplify code with `String.join()` LGTM, however are there tests that ensure the changes are benign? - PR: https://git.openjdk.org/jdk/pull/9513

Case Insensitive DateFormatSymbols for parsing

2022-07-15 Thread Bernd Eckenfels
Hello, I noticed that it is surprisingly hard to make SimpleDateFormat accept all-uppercase month names while parsing. Even with a custom DateFormatSymbols that’s hard because you can only specify a single symbol for a month name. For parsing it would be good if you can either specify a list

RFR: 8290300: Use standard String-joining tools where applicable

2022-07-15 Thread Сергей Цыпанов
Simplify code with `String.join()` - Commit messages: - 8290300: Fix copy-right year - 8290300: Use standard String-joining tools where applicable Changes: https://git.openjdk.org/jdk/pull/9513/files Webrev: https://webrevs.openjdk.org/?repo=jdk=9513=00 Issue:

Re: RFR: 8286849: Use @Stable for generic repositories

2022-07-15 Thread liach
On Tue, 17 May 2022 04:40:50 GMT, liach wrote: > Generic repositories, the implementation detail for generic information in > core reflection, can be updated to use the `@Stable` annotation to replace > their `volatile` access. Their existing accessor code is already safe, > reading the cache

Re: RFR: 8290327: Remove java/lang/reflect/callerCache/ReflectionCallerCacheTest.java from ProblemList-Xcomp.txt

2022-07-15 Thread Jaikiran Pai
On Thu, 14 Jul 2022 21:01:05 GMT, Mandy Chung wrote: > It's expected that [JDK-8287596](https://bugs.openjdk.org/browse/JDK-8287596) > has resolved [JDK-8288286](https://bugs.openjdk.org/browse/JDK-8288286). > Remove ReflectionCallerCacheTest.java from the problem list for test > execution.

RFR: 8289471: Issue in Initialization of keys in ErrorMsg.java and XPATHErrorResources.java

2022-07-15 Thread Shruthi Shruthi1
OpenJDK bug: JDK-8289471 PR: https://github.com/openjdk/jdk/pull/9369

Integrated: 8028265: Add legacy tz tests to OpenJDK

2022-07-15 Thread Yoshiki Sato
On Wed, 13 Jul 2022 04:30:39 GMT, Yoshiki Sato wrote: > Please review this PR. The PR open sources the closed timezone tests. This pull request has now been integrated. Changeset: 92deab54 Author:Yoshiki Sato Committer: Sean Coffey URL:

Re: RFR: 8290327: Remove java/lang/reflect/callerCache/ReflectionCallerCacheTest.java from ProblemList-Xcomp.txt

2022-07-15 Thread Jaikiran Pai
On Thu, 14 Jul 2022 21:01:05 GMT, Mandy Chung wrote: > It's expected that [JDK-8287596](https://bugs.openjdk.org/browse/JDK-8287596) > has resolved [JDK-8288286](https://bugs.openjdk.org/browse/JDK-8288286). > Remove ReflectionCallerCacheTest.java from the problem list for test > execution.