Integrated: 8300644: Remove gc/shenandoah/jni/TestStringCriticalWithDedup.java

Erik Österlund eosterlund at openjdk.org
Wed Jan 25 08:19:12 UTC 2023


On Thu, 19 Jan 2023 10:26:38 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:

> The gc/shenandoah/jni/TestStringCriticalWithDedup.java test was designed to catch failure to pin strings being passed out to JNI critical users, because that used to be dangerous. After [JDK-8299673](https://bugs.openjdk.org/browse/JDK-8299673) that is not dangerous any longer. Conversely, now we kind of want deduplication to proceed regardless of JNI critical, which defeats the purpose of this test. It should be removed.

This pull request has now been integrated.

Changeset: 95fafd09
Author:    Erik Österlund <eosterlund at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/95fafd094f93eaf3ff15c76ca25345123d1586fe
Stats:     206 lines in 2 files changed: 0 ins; 206 del; 0 mod

8300644: Remove gc/shenandoah/jni/TestStringCriticalWithDedup.java

Reviewed-by: wkemper, mbaesken

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

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


More information about the shenandoah-dev mailing list