On Sun, 18 May 2025 21:58:04 GMT, Chen Liang <li...@openjdk.org> wrote:

> The test was originally written with product builds in mind; it didn't 
> consider the slowness of debug builds, thus causing tests to fail.
> 
> Also there was a bug in testWeakAgainstClassValue: the test need to flush the 
> backing map in the class to have the weak value removed. It is fixed in this 
> patch as well.
> 
> Testing: ClassValueTest repeated 10 times on 
> windows-x64/linux-x64/macosx-aarch64/linux-aarch64 fastdebug.

I still wish to keep these explicit timeouts - they can be fiddled around for 
debugging when something stalls after a ClassValue change. In comparison, the 
argless variants are much harder to tweak and/or debug.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/25292#issuecomment-2898217956

Reply via email to