Integrated: 8364927: Add @requires annotation to TestReclaimStringsLeaksMemory.java
jonghoonpark
duke at openjdk.org
Mon Oct 6 11:56:00 UTC 2025
On Wed, 13 Aug 2025 19:11:25 GMT, jonghoonpark <duke at openjdk.org> wrote:
> This change updates `TestReclaimStringsLeaksMemory.java` to properly guard against unsupported GC arguments.
> Previously, the test only checked `@requires vm.gc == "null"`.
>
> The fix separates the tests and adds explicit `@requires` clauses for each GC configuration to ensure the test only runs when supported.
>
> This prevents test failures on VMs that do not support the specified GC.
>
> (This is my first contribution to the JDK. Thank you for your review and guidance.)
This pull request has now been integrated.
Changeset: 2bfada3f
Author: jonghoonpark <dev at jonghoonpark.com>
Committer: Thomas Schatzl <tschatzl at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/2bfada3f58df6c041d948267368cbc4db915cac3
Stats: 46 lines in 1 file changed: 42 ins; 1 del; 3 mod
8364927: Add @requires annotation to TestReclaimStringsLeaksMemory.java
Reviewed-by: tschatzl, stefank, ayang
-------------
PR: https://git.openjdk.org/jdk/pull/26766
More information about the hotspot-gc-dev
mailing list