On Thu, 13 Mar 2025 19:01:03 GMT, Joe Wang wrote:
> Add public identifiers to the JDK built-in Catalog; Replace the incorrect
> Schema 1.1 DTD files (note the Public Identifier at line 2) with the correct
> Shema 1.0 DTDs.
Looks good to me, just a few very minor questions. Thank you
test/jaxp
On Thu, 20 Mar 2025 22:35:23 GMT, Brent Christian wrote:
> I propose some cleanups to `FinalizerHistogramTest.java` to hopefully clear
> up the intermittent failures:
>
> * run with `othervm`: this test blocks the (global) finalizer thread, and
> also requires the (global) finalizer thread to
Updating `jdk.test.lib.SecurityTools` class to have jar method accept
`List` parameter similar to `keytool` methods.
Currently this only accept String parameter, which is error prone with missing
space
-
Commit messages:
- Update jdk.test.lib.SecurityTools jar method to accept List
On Fri, 14 Mar 2025 18:35:12 GMT, Joe Wang wrote:
> Thanks for review. Updated the test.
Thank you!
-
PR Comment: https://git.openjdk.org/jdk/pull/24039#issuecomment-2729299732
On Fri, 21 Mar 2025 09:01:01 GMT, KIRIYAMA Takuya wrote:
>> test/jdk/java/lang/RuntimeTests/RuntimeExitLogTest.java line 52:
>>
>>> 50: private static final String TEST_JDK =
>>> System.getProperty("test.jdk");
>>> 51: private static final String TEST_SRC =
>>> System.getProperty("test
On Fri, 14 Mar 2025 09:17:06 GMT, KIRIYAMA Takuya wrote:
> The current test program for the logging feature added in JDK-8301627 does
> not fully check some important cases.
>
> Issue Details:
> The test does not properly check cases where logging might not happen due to
> different logging le
On Tue, 25 Mar 2025 11:56:37 GMT, Mikhail Yankelevich
wrote:
> Updating `jdk.test.lib.SecurityTools` class to have jar method accept
> `List` parameter similar to `keytool` methods.
> Currently this only accept String parameter, which is error prone with
> missing space
This pull
On Tue, 22 Apr 2025 09:05:09 GMT, Jaikiran Pai wrote:
> Can I please get a review of this test only change which proposes to improve
> the debuggability of `com/sun/jndi/ldap/LdapPoolTimeoutTest.java`?
>
> This test has been failing intermittently in our CI for several years.
> Detailed are no
On Tue, 22 Apr 2025 09:05:09 GMT, Jaikiran Pai wrote:
> Can I please get a review of this test only change which proposes to improve
> the debuggability of `com/sun/jndi/ldap/LdapPoolTimeoutTest.java`?
>
> This test has been failing intermittently in our CI for several years.
> Detailed are no
On Tue, 22 Apr 2025 11:08:15 GMT, Jaikiran Pai wrote:
>> Can I please get a review of this test only change which proposes to improve
>> the debuggability of `com/sun/jndi/ldap/LdapPoolTimeoutTest.java`?
>>
>> This test has been failing intermittently in our CI for several years.
>> Detailed a
On Wed, 16 Apr 2025 00:42:22 GMT, Brian Burkhalter wrote:
>> In `java.io.WinNTFileSystem::isInvalid`, replace an insufficient test for
>> file path validity with a sufficient test for file path invalidity. Also,
>> add a new test.
>
> Brian Burkhalter has updated the pull request incrementally
On Wed, 30 Apr 2025 20:17:57 GMT, Alexander Matveev
wrote:
> - Symbolic links will not be followed for `--app-content` on all platforms.
> - Added test to cover this case.
> - `MacHelper` updated to use "cp -R" instead of "cp -r" when unpacking DMG,
> since "cp -r" on macOS is not documented op
On Mon, 3 Mar 2025 20:24:54 GMT, Vladimir Ivanov wrote:
> test setup was updated to generate data of requested size.
Looks good. Just 1 minor question
test/micro/org/openjdk/bench/java/lang/foreign/ToCStringTest.java line 100:
> 98: mollit anim id est laborum.
> 99:
On Tue, 25 Feb 2025 23:50:01 GMT, Chen Liang wrote:
> Core reflection's generic signature parsing system is used for many aspects,
> including annotations and enclosing methods, yet it is under-tested. It is
> better for us to set up tests to ensure that sensitive error behaviors of
> core ref
On Thu, 22 May 2025 14:19:09 GMT, Viktor Klang wrote:
> Barring other effects, it is likely that this test just needs to have some
> longer timeouts to ensure that a stall isn't likely to fail the test case.
> The shouldn't have any impact on test execution duration, as the test looks
> at tas
On Tue, 8 Jul 2025 16:10:17 GMT, Darragh Conway wrote:
> Refactored extract method to encapsulate Windows specific test logic
Looks good, just a minor comment
test/jdk/java/io/File/MaxPathLength.java line 200:
> 198: String name = fileName;
> 199: while (name.length() <
On Wed, 9 Jul 2025 12:11:10 GMT, Darragh Conway wrote:
>> test/jdk/java/io/File/MaxPathLength.java line 202:
>>
>>> 200: testLongPath (20, name, false);
>>> 201: testLongPath (20, name, true);
>>> 202: name = getNextName(name);
>>
>> Name doesn't
On Mon, 21 Jul 2025 13:28:00 GMT, Volkan Yazici wrote:
>> Validate input in `java.lang.StringCoding` intrinsic Java wrappers, improve
>> their documentation, enhance the checks in the associated IR or assembly
>> code, and adapt them to cause VM crash on invalid input.
>>
>> ## Implementation
On Sun, 20 Jul 2025 22:19:07 GMT, simon wrote:
> 8355652: add new method to return ClassFileFormatVersion from
> ClassFileVersion.
> -
> ### Progress
> - [ ] Change must be properly reviewed (1 review required, with at least 1
> [Reviewer](https://openjdk.org/bylaws#reviewer))
> - [x] C
On Thu, 24 Jul 2025 14:50:07 GMT, Tatsunori Uchino wrote:
> Adds `codePointCount()` overloads to `String`, `Character`,
> `(Abstract)StringBuilder`, and `StringBuffer` to make it possible to
> conveniently retrieve the length of a string as code points without extra
> boundary checks.
>
>
>
On Thu, 24 Jul 2025 14:50:07 GMT, Tatsunori Uchino wrote:
> Adds `codePointCount()` overloads to `String`, `Character`,
> `(Abstract)StringBuilder`, and `StringBuffer` to make it possible to
> conveniently retrieve the length of a string as code points without extra
> boundary checks.
>
>
>
On Tue, 22 Jul 2025 11:21:46 GMT, fabioromano1 wrote:
>> This PR implements nth root computation for BigIntegers using Newton method.
>
> fabioromano1 has updated the pull request incrementally with one additional
> commit since the last revision:
>
> No need to check for root's shift overflo
22 matches
Mail list logo