Skip to content

Commit 6b838fc

Browse files
committedOct 27, 2020
rename permittedSubclasses to getPermittedSubclasses
1 parent 3999025 commit 6b838fc

File tree

4 files changed

+12
-12
lines changed

4 files changed

+12
-12
lines changed
 

‎src/java.base/share/classes/java/lang/Class.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -4398,7 +4398,7 @@ public Optional<ClassDesc> describeConstable() {
43984398
* @since 15
43994399
*/
44004400
@jdk.internal.PreviewFeature(feature=jdk.internal.PreviewFeature.Feature.SEALED_CLASSES, essentialAPI=false)
4401-
public Class<?>[] permittedSubclasses() {
4401+
public Class<?>[] getPermittedSubclasses() {
44024402
Class<?>[] subClasses;
44034403
if (isArray() || isPrimitive() || (subClasses = getPermittedSubclasses0()).length == 0) {
44044404
return EMPTY_CLASS_ARRAY;
@@ -4435,7 +4435,7 @@ public boolean isSealed() {
44354435
if (isArray() || isPrimitive()) {
44364436
return false;
44374437
}
4438-
return permittedSubclasses().length != 0;
4438+
return getPermittedSubclasses().length != 0;
44394439
}
44404440

44414441
private native Class<?>[] getPermittedSubclasses0();

‎test/hotspot/jtreg/runtime/sealedClasses/GetPermittedSubclassesTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ final class NotSealed implements SealedI1 {}
5050
final class Final4 {}
5151

5252
public static void testSealedInfo(Class<?> c, String[] expected) {
53-
Object[] permitted = c.permittedSubclasses();
53+
Object[] permitted = c.getPermittedSubclasses();
5454

5555
if (permitted.length != expected.length) {
5656
throw new RuntimeException(

‎test/jdk/java/lang/reflect/sealed_classes/SealedClassesReflectionTest.java

+8-8
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,8 @@ public Object[][] sealedClassesData() {
8686
public void testSealedClasses(Class<?> cls) {
8787
assertTrue(cls.isSealed());
8888
assertTrue(!Modifier.isFinal(cls.getModifiers()));
89-
assertTrue(cls.permittedSubclasses() != null);
90-
assertTrue(cls.permittedSubclasses().length > 0);
89+
assertTrue(cls.getPermittedSubclasses() != null);
90+
assertTrue(cls.getPermittedSubclasses().length > 0);
9191
}
9292

9393
@DataProvider(name = "notSealedClasses")
@@ -110,8 +110,8 @@ public Object[][] notSealedClassesData() {
110110
@Test(dataProvider = "notSealedClasses")
111111
public void testNotSealedClasses(Class<?> cls) {
112112
assertTrue(!cls.isSealed());
113-
assertTrue(cls.permittedSubclasses() != null);
114-
assertTrue(cls.permittedSubclasses().length == 0);
113+
assertTrue(cls.getPermittedSubclasses() != null);
114+
assertTrue(cls.getPermittedSubclasses().length == 0);
115115
}
116116

117117
@DataProvider(name = "non_sealedClasses")
@@ -128,8 +128,8 @@ public void testnon_sealedClasses(Class<?> cls) {
128128
assertTrue(!cls.isSealed());
129129
assertTrue(!Modifier.isFinal(cls.getModifiers()));
130130
assertTrue((cls.getSuperclass() != null && cls.getSuperclass().isSealed()) || Arrays.stream(cls.getInterfaces()).anyMatch(Class::isSealed));
131-
assertTrue(cls.permittedSubclasses() != null);
132-
assertTrue(cls.permittedSubclasses().length == 0);
131+
assertTrue(cls.getPermittedSubclasses() != null);
132+
assertTrue(cls.getPermittedSubclasses().length == 0);
133133
}
134134

135135
@DataProvider(name = "reflectionData")
@@ -215,9 +215,9 @@ public void testSealedReflection(Class<?> sealedClass,
215215
throws ReflectiveOperationException
216216
{
217217
assertTrue(sealedClass.isSealed());
218-
assertTrue(sealedClass.permittedSubclasses().length == numberOfSubclasses);
218+
assertTrue(sealedClass.getPermittedSubclasses().length == numberOfSubclasses);
219219
int i = 0;
220-
for (Class<?> cd : sealedClass.permittedSubclasses()) {
220+
for (Class<?> cd : sealedClass.getPermittedSubclasses()) {
221221
assertTrue(cd.getName().equals(subclassDescriptors[i]), "expected: " + subclassDescriptors[i] + " found: " + cd.getName());
222222
i++;
223223
}

‎test/langtools/tools/javac/sealed/SealedCompilationTests.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -613,7 +613,7 @@ public void testAPIForPrimitiveAndArrayClasses() {
613613
float.class, float[].class, double.class, double[].class, char.class, char[].class, boolean.class, boolean[].class, void.class,
614614
String[].class}) {
615615
Assert.check(!c.isSealed());
616-
Assert.check(c.permittedSubclasses().length == 0);
616+
Assert.check(c.getPermittedSubclasses().length == 0);
617617
}
618618
}
619619

0 commit comments

Comments
 (0)
Please sign in to comment.