clear the AccAbstract and the AccInterface bits

subyssurendran666 · Jan 6, 2025 · 4b39e64 · 4b39e64
1 parent 0e950f4
commit 4b39e64
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/org.eclipse.jdt.core.tests.model/src/org/eclipse/jdt/core/tests/dom/ASTConverter_23Test.java b/org.eclipse.jdt.core.tests.model/src/org/eclipse/jdt/core/tests/dom/ASTConverter_23Test.java
@@ -462,6 +462,7 @@ public final  class X implements A {}
 	    ITypeBinding aBinding = a.resolveBinding();
 	    assertTrue(Modifier.isSealed(aBinding.getModifiers()));
 	    assertTrue(Modifier.isPublic(aBinding.getModifiers()));
+	    assertFalse(Modifier.isNonSealed(aBinding.getModifiers()));
 	}
 
 	//public non-sealed interface
@@ -494,6 +495,7 @@ public final  class X implements A {}
 	    ITypeBinding aBinding = a.resolveBinding();
 	    assertTrue(Modifier.isNonSealed(aBinding.getModifiers()));
 	    assertTrue(Modifier.isPublic(aBinding.getModifiers()));
+	    assertFalse(Modifier.isSealed(aBinding.getModifiers()));
 	}
 
 	//public strictfp interface

diff --git a/org.eclipse.jdt.core/dom/org/eclipse/jdt/core/dom/TypeBinding.java b/org.eclipse.jdt.core/dom/org/eclipse/jdt/core/dom/TypeBinding.java
@@ -607,7 +607,9 @@ public int getModifiers() {
 			return accessFlags & ~(ClassFileConstants.AccAbstract | ClassFileConstants.AccInterface | ClassFileConstants.AccAnnotation);
 		} else if (isInterface()) {
 			ReferenceBinding referenceBinding = (ReferenceBinding) this.binding;
-			final int accessFlags = referenceBinding.getAccessFlags() & VALID_MODIFIERS;
+			int accessFlags = referenceBinding.getAccessFlags() & VALID_MODIFIERS;
+			// clear the AccAbstract and the AccInterface bits
+			accessFlags = accessFlags & ~(ClassFileConstants.AccAbstract | ClassFileConstants.AccInterface);
 
 			if (referenceBinding.isSealed()) {
 				return accessFlags | Modifier.SEALED;
@@ -616,7 +618,7 @@ public int getModifiers() {
 				return accessFlags | Modifier.NON_SEALED;
 			}
 			// clear the AccAbstract and the AccInterface bits
-			return accessFlags & ~(ClassFileConstants.AccAbstract | ClassFileConstants.AccInterface);
+			return accessFlags;
 		} else if (isEnum()) {
 			ReferenceBinding referenceBinding = (ReferenceBinding) this.binding;
 			final int accessFlags = referenceBinding.getAccessFlags() & VALID_MODIFIERS;