Re: RFR: 8291429: java/lang/Thread/virtual/ThreadAPI.java timed out on single core system [v2]

2022-10-06 Thread Chris Plummer
On Thu, 6 Oct 2022 13:39:25 GMT, Alan Bateman wrote: >> This is a test only change for two tests for virtual threads that >> hang/timeout on single core systems. The two tests involve pinning and >> require at least two carrier threads. The test lib used by these tests is >> updated to define

Integrated: 8293810: Remove granting of RuntimePermission("stopThread") from tests

2022-10-06 Thread Alan Bateman
On Wed, 5 Oct 2022 15:01:15 GMT, Alan Bateman wrote: > This is a test only change to remove the granting of > RuntimePermission("stopThread") from the tests. With Thread.stop changed to > throw UOE it means there is nothing that requires this permission. This pull request has now been integrat

Integrated: 6560981: (cal) unused local variables in GregorianCalendar, etc.

2022-10-06 Thread Justin Lu
On Wed, 5 Oct 2022 22:17:26 GMT, Justin Lu wrote: > Problem: Unused variables in GregorianCalendar, JapaneseImperialCalendar, and > Base Calendar. > > Fix: Removed all unused variables in bug description except normalizedYear in > JapaneseImpericalCalendar.getActualMaximum.() as there was no m

Re: RFR: 6560981: (cal) unused local variables in GregorianCalendar, etc.

2022-10-06 Thread Brent Christian
On Wed, 5 Oct 2022 22:17:26 GMT, Justin Lu wrote: > Problem: Unused variables in GregorianCalendar, JapaneseImperialCalendar, and > Base Calendar. > > Fix: Removed all unused variables in bug description except normalizedYear in > JapaneseImpericalCalendar.getActualMaximum.() as there was no m

Re: RFR: 6560981: (cal) unused local variables in GregorianCalendar, etc.

2022-10-06 Thread Iris Clark
On Wed, 5 Oct 2022 22:17:26 GMT, Justin Lu wrote: > Problem: Unused variables in GregorianCalendar, JapaneseImperialCalendar, and > Base Calendar. > > Fix: Removed all unused variables in bug description except normalizedYear in > JapaneseImpericalCalendar.getActualMaximum.() as there was no m

Re: RFR: 6560981: (cal) unused local variables in GregorianCalendar, etc.

2022-10-06 Thread Naoto Sato
On Wed, 5 Oct 2022 22:17:26 GMT, Justin Lu wrote: > Problem: Unused variables in GregorianCalendar, JapaneseImperialCalendar, and > Base Calendar. > > Fix: Removed all unused variables in bug description except normalizedYear in > JapaneseImpericalCalendar.getActualMaximum.() as there was no m

RFR: 6560981: (cal) unused local variables in GregorianCalendar, etc.

2022-10-06 Thread Justin Lu
Problem: Unused variables in GregorianCalendar, JapaneseImperialCalendar, and Base Calendar. Fix: Removed all unused variables in bug description except normalizedYear in JapaneseImpericalCalendar.getActualMaximum.() as there was no matching unused variable within that method. Additionally rem

Re: RFR: 8170389: java.text.DigitList.getDouble() : Controversy between javadoc and code [v4]

2022-10-06 Thread Lance Andersen
On Thu, 6 Oct 2022 20:43:30 GMT, Justin Lu wrote: >> Problem: Outdated doc does not match code. Claimed to throw exception and >> compared to Long method. >> Fix: Update doc to match code, compared to Double.parseDouble() accordingly. > > Justin Lu has updated the pull request incrementally with

Re: RFR: 8170389: java.text.DigitList.getDouble() : Controversy between javadoc and code [v4]

2022-10-06 Thread Justin Lu
> Problem: Outdated doc does not match code. Claimed to throw exception and > compared to Long method. > Fix: Update doc to match code, compared to Double.parseDouble() accordingly. Justin Lu has updated the pull request incrementally with one additional commit since the last revision: Refact

Re: RFR: 8170389: java.text.DigitList.getDouble() : Controversy between javadoc and code [v3]

2022-10-06 Thread Justin Lu
On Thu, 6 Oct 2022 10:58:36 GMT, Lance Andersen wrote: > Looks fine Justin. > > Please make sure the comments are < 80 characters and if they are longer > break them into two lines as they seem like they might be long @LanceAndersen Made the change, thanks Lance - PR: https://git

Re: RFR: 8291429: java/lang/Thread/virtual/ThreadAPI.java timed out on single core system [v2]

2022-10-06 Thread Alan Bateman
On Thu, 6 Oct 2022 15:31:02 GMT, Chris Plummer wrote: > If these tests are ever run with the virtual thread wrapper, will we end up > being short a carrier thread? It's unclear to me if the tests will always > have at least one unpinned carrier thread to work with. There's nothing inherit here

Re: RFR: 8292177: InitialSecurityProperty JFR event [v3]

2022-10-06 Thread Sean Mullan
On Mon, 3 Oct 2022 10:30:54 GMT, Sean Coffey wrote: >> New JFR event to record state of initial security properties. >> >> Debug output is also now added for these properties via >> -Djava.security.debug=properties > > Sean Coffey has updated the pull request incrementally with one additional

Re: RFR: 8290368: Introduce LDAP and RMI protocol-specific object factory filters to JNDI implementation

2022-10-06 Thread Roger Riggs
On Wed, 5 Oct 2022 15:23:43 GMT, Aleksei Efimov wrote: > ### Summary of the change > This change introduces new system and security properties for specifying > factory filters for the JNDI/LDAP and the JNDI/RMI JDK provider > implementations. > > These new properties allow more granular contr

Re: RFR: 8291429: java/lang/Thread/virtual/ThreadAPI.java timed out on single core system [v2]

2022-10-06 Thread Chris Plummer
On Thu, 6 Oct 2022 13:39:25 GMT, Alan Bateman wrote: >> This is a test only change for two tests for virtual threads that >> hang/timeout on single core systems. The two tests involve pinning and >> require at least two carrier threads. The test lib used by these tests is >> updated to define

Re: RFR: 8294377: Prepare to stop auto-inheriting documentation for subclasses of exceptions whose documentation is inherited [v2]

2022-10-06 Thread Jonathan Gibbons
On Tue, 27 Sep 2022 12:14:23 GMT, Pavel Rappo wrote: >> This adds exception documentation to JDK methods that would otherwise lose >> that documentation once JDK-8287796 is integrated. While adding this >> exception documentation now does not change [^1] the JDK API Documentation, >> it will a

Re: RFR: 8291429: java/lang/Thread/virtual/ThreadAPI.java timed out on single core system [v2]

2022-10-06 Thread Alan Bateman
On Thu, 6 Oct 2022 14:26:27 GMT, Serguei Spitsyn wrote: > Just a side note to be aware about a potential problem. > This won't work correctly if there are more than one such tests executed in > VM at the same time. > Then there can be a race to restore the parallelism value. > But it seems is no

Re: RFR: 8294377: Prepare to stop auto-inheriting documentation for subclasses of exceptions whose documentation is inherited [v2]

2022-10-06 Thread Pavel Rappo
On Fri, 30 Sep 2022 19:15:17 GMT, Phil Race wrote: > If the docs end up the same as you say that is fine by me since for the two > ImageIO classes that seems to be what we'd want. Since this change does not affect HTML pages for java.desktop, I take it as "approved". > But inheritDoc behaviou

Re: RFR: 8291429: java/lang/Thread/virtual/ThreadAPI.java timed out on single core system [v2]

2022-10-06 Thread Serguei Spitsyn
On Thu, 6 Oct 2022 13:39:25 GMT, Alan Bateman wrote: >> This is a test only change for two tests for virtual threads that >> hang/timeout on single core systems. The two tests involve pinning and >> require at least two carrier threads. The test lib used by these tests is >> updated to define

Re: RFR: 8282664: Unroll by hand StringUTF16, StringLatin1, and Arrays polynomial hash loops [v14]

2022-10-06 Thread Claes Redestad
On Tue, 14 Jun 2022 13:54:46 GMT, Ludovic Henry wrote: >> Ludovic Henry has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Reenable SpecialArraysHashCode by default > > Still working on it, other work priorities have popped up. I'm taking t

Re: RFR: 8291429: java/lang/Thread/virtual/ThreadAPI.java timed out on single core system [v2]

2022-10-06 Thread Alan Bateman
On Wed, 5 Oct 2022 20:24:59 GMT, Chris Plummer wrote: >> No, restoration requires a call to close. Maybe you are assuming that >> AutoCloseable implementations setup a cleaner to do that? > > Yes, that was my point of confusion. I thought collection triggered calling > close, but I see now when

Re: RFR: 8291429: java/lang/Thread/virtual/ThreadAPI.java timed out on single core system [v2]

2022-10-06 Thread Alan Bateman
> This is a test only change for two tests for virtual threads that > hang/timeout on single core systems. The two tests involve pinning and > require at least two carrier threads. The test lib used by these tests is > updated to define a new method that ensures parallelism is at least a given

Re: RFR: 8291917: Windows - Improve error messages when the C Runtime Libraries or jvm.dll cannot be loaded [v13]

2022-10-06 Thread Julian Waters
On Wed, 5 Oct 2022 16:40:16 GMT, Julian Waters wrote: >> Please review a small patch for dumping the failure reason when the MSVCRT >> libraries or the Java Virtual Machine fails to load on Windows, which can >> provide invaluable insight when debugging related launcher issues. >> >> See https

Re: RFR: 8291917: Windows - Improve error messages when the C Runtime Libraries or jvm.dll cannot be loaded [v14]

2022-10-06 Thread Julian Waters
> Please review a small patch for dumping the failure reason when the MSVCRT > libraries or the Java Virtual Machine fails to load on Windows, which can > provide invaluable insight when debugging related launcher issues. > > See https://bugs.openjdk.org/browse/JDK-8292016 and the related Pull R

Re: RFR: 8293810: Remove granting of RuntimePermission("stopThread") from tests

2022-10-06 Thread Chris Hegarty
On Wed, 5 Oct 2022 15:01:15 GMT, Alan Bateman wrote: > This is a test only change to remove the granting of > RuntimePermission("stopThread") from the tests. With Thread.stop changed to > throw UOE it means there is nothing that requires this permission. LGTM - Marked as reviewed

Re: RFR: 8290368: Introduce LDAP and RMI protocol-specific object factory filters to JNDI implementation

2022-10-06 Thread Daniel Fuchs
On Wed, 5 Oct 2022 15:23:43 GMT, Aleksei Efimov wrote: > ### Summary of the change > This change introduces new system and security properties for specifying > factory filters for the JNDI/LDAP and the JNDI/RMI JDK provider > implementations. > > These new properties allow more granular contr

Re: RFR: 8170389: java.text.DigitList.getDouble() : Controversy between javadoc and code [v3]

2022-10-06 Thread Lance Andersen
On Wed, 5 Oct 2022 21:26:09 GMT, Justin Lu wrote: >> Problem: Outdated doc does not match code. Claimed to throw exception and >> compared to Long method. >> Fix: Update doc to match code, compared to Double.parseDouble() accordingly. > > Justin Lu has updated the pull request incrementally with

Integrated: 8294456: Fix misleading-indentation warnings in core JDK libraries

2022-10-06 Thread Raffaello Giulietti
On Thu, 29 Sep 2022 13:11:03 GMT, Raffaello Giulietti wrote: > This fixes misleading indentations, which allows enabling the (currently > disabled) `misleading-indentation` warning flag on two `.gmk` files. This pull request has now been integrated. Changeset: 2ceebf68 Author:Raffaello Gi

Integrated: 8294697: java/lang/Thread/virtual/ThreadAPI.testGetStackTrace2 failed with non-empty stack trace

2022-10-06 Thread Alan Bateman
On Wed, 5 Oct 2022 14:05:38 GMT, Alan Bateman wrote: > This is a test-only change. ThreadAPI.testGetStackTrace2 tests the stack > trace of a virtual thread that has started but has not executed it. The test > currently attempts to saturate all carrier threads before starting a target > thread.

Integrated: JDK-8294837: unify Windows 2019 version check in os_windows and java_props_md

2022-10-06 Thread Matthias Baesken
On Wed, 5 Oct 2022 09:21:50 GMT, Matthias Baesken wrote: > Currently the buildNumber check for Windows 2019 server differs in > os_windows.cpp and java_props_md.c ( java_props_md.c still checks pre GA > versions , this is probably not necessary any more ). > The check should be unified. This p

Re: RFR: JDK-8294837: unify Windows 2019 version check in os_windows and java_props_md

2022-10-06 Thread Matthias Baesken
On Wed, 5 Oct 2022 09:21:50 GMT, Matthias Baesken wrote: > Currently the buildNumber check for Windows 2019 server differs in > os_windows.cpp and java_props_md.c ( java_props_md.c still checks pre GA > versions , this is probably not necessary any more ). > The check should be unified. Hi Rog