RFR: 8287596: Reorg jdk.test.lib.util.ForceGC [v9]

Roger Riggs rriggs at openjdk.org
Thu Jun 16 16:11:56 UTC 2022


On Thu, 16 Jun 2022 15:55:00 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

>> This is a follow up update per comments in [JDK-8287384 PR](https://urldefense.com/v3/__https://github.com/openjdk/jdk/pull/8907__;!!ACWV5N9M2RV99hQ!LTtncpyN_2kanJmZF3wz3O88hB5AwboRKEitwQ0UJpiaZQ7RhIZpzgU4kCL3gwrt_JRqji2j5Tb92sOTWGguKw$ ).  The tier1 and tier2 test in open part looks good to me.  Please help to run Mach5 just case the closed test cases are impacted.
>
> Xue-Lei Andrew Fan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   remove trailing whitespaces

test/jdk/java/io/ObjectStreamClass/TestOSCClassLoaderLeak.java line 64:

> 62:         assertNotNull(myOwnClassLoaderWeakReference.get());
> 63: 
> 64:         assertTrue(ForceGC.wait(() -> myOwnClassLoaderWeakReference.get() == null));

The call to ref.get() can create a strong reference to the object; depending on the timing it might interfere with the GC in process.
`ref.refersTo(null)` is preferred over `ref.get() == null` 

Here and all subsequent changes.

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

PR: https://git.openjdk.org/jdk/pull/8979



More information about the security-dev mailing list