Simplify code per review comments.

openjdk · May 18, 2020 · ecb41ea · ecb41ea
1 parent 94e232e
commit ecb41ea
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 14 deletions.
diff --git a/src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symtab.java b/src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symtab.java
@@ -271,14 +271,8 @@ public VarSymbol getClassField(Type type, Types types) {
                        I1, I2, ... In, V.class is typed to be Class<? extends Object & I1 & I2 .. & In>
                     */
                     if (type.isValue()) {
-                        ClassType ct = (ClassType) type;
-                        Type it;
-                        if (ct.interfaces_field == null || ct.interfaces_field.isEmpty()) {
-                            it = objectType;
-                        } else {
-                            it = types.makeIntersectionType(ct.interfaces_field, true);
-                        }
-                        arg = new WildcardType(it, BoundKind.EXTENDS, boundClass);
+                        List<Type> bounds = List.of(objectType).appendList(((ClassSymbol) type.tsym).getInterfaces());
+                        arg = new WildcardType(types.makeIntersectionType(bounds), BoundKind.EXTENDS, boundClass);
                     } else {
                         arg = types.erasure(type);
                     }

diff --git a/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java b/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java
@@ -2522,12 +2522,8 @@ Type adjustMethodReturnType(Symbol msym, Type qualifierType, Name methodName, Li
                 // I1, I2, ... In, v.getClass() is typed to be Class<? extends Object & I1 & I2 .. & In>
                 Type wcb;
                 if (qualifierType.isValue()) {
-                    ClassType ct = (ClassType) qualifierType;
-                    if (ct.interfaces_field == null || ct.interfaces_field.isEmpty()) {
-                        wcb = syms.objectType;
-                    } else {
-                        wcb = types.makeIntersectionType(ct.interfaces_field, true);
-                    }
+                    List<Type> bounds = List.of(syms.objectType).appendList(((ClassSymbol) qualifierType.tsym).getInterfaces());
+                    wcb = types.makeIntersectionType(bounds);
                 } else {
                     wcb = types.erasure(qualifierType);
                 }