From: aclement <aclement>
Date: Thu, 26 Feb 2004 13:34:28 +0000 (+0000)
Subject: Fix for Bugzilla Bug 52107
X-Git-Tag: v_preCompileLoopAlteration~16
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=31bf65adc6905c7b6922a6d2b15e836267f33151;p=aspectj.git

Fix for Bugzilla Bug 52107
   NoSuchMethodError accessing field declared on interface
---

diff --git a/tests/ajcTests.xml b/tests/ajcTests.xml
index 2081e9e7c..35b7d15f9 100644
--- a/tests/ajcTests.xml
+++ b/tests/ajcTests.xml
@@ -7236,4 +7236,31 @@
         </compile>
     </ajc-test>
     
+   	<ajc-test dir="bugs/fieldsOnInterfaces"
+		pr="52107"
+		title="declare String field on interface">
+		<compile files="StringFieldOnInterface.java">
+			<message kind="warning" line="16" text="this affected type is not exposed"/>
+         	<message kind="warning" line="17" text="this affected type is not exposed"/>
+		</compile>	   
+	</ajc-test>
+
+	<ajc-test dir="bugs/fieldsOnInterfaces"
+		pr="52107"
+		title="declare int field on interface">
+		<compile files="IntFieldOnInterface.java">
+		<message kind="warning" line="27" text="this affected type is not exposed"/>
+         	<message kind="warning" line="28" text="this affected type is not exposed"/>
+		</compile>
+	</ajc-test>
+	
+	<ajc-test dir="bugs/fieldsOnInterfaces"
+		pr="52107"
+		title="declare Object field on interface">
+		<compile files="ObjectFieldOnInterface.java">
+			<message kind="warning" line="20" text="this affected type is not exposed"/>
+         	<message kind="warning" line="21" text="this affected type is not exposed"/>
+		</compile>
+	</ajc-test>
+    
 </suite>
diff --git a/tests/ajcTestsFailing.xml b/tests/ajcTestsFailing.xml
index cd8ba3e47..c2fb46b05 100644
--- a/tests/ajcTestsFailing.xml
+++ b/tests/ajcTestsFailing.xml
@@ -133,25 +133,4 @@
         <run class="InterfaceInitializerOrder"/>
     </ajc-test>
 
-	<ajc-test dir="bugs/fieldsOnInterfaces"
-		pr="52107"
-		title="declare String field on interface">
-		<compile files="StringFieldOnInterface.java"/>
-		<run class="StringFieldOnInterface"/>
-		</ajc-test>
-
-	<ajc-test dir="bugs/fieldsOnInterfaces"
-		pr="52107"
-		title="declare int field on interface">
-		<compile files="IntFieldOnInterface.java"/>
-		<run class="IntFieldOnInterface"/>
-		</ajc-test>
-	
-	<ajc-test dir="bugs/fieldsOnInterfaces"
-		pr="52107"
-		title="declare Object field on interface">
-		<compile files="ObjectFieldOnInterface.java"/>
-		<run class="ObjectFieldOnInterface"/>
-		</ajc-test>
-
 </suite>
diff --git a/weaver/src/org/aspectj/weaver/ResolvedTypeMunger.java b/weaver/src/org/aspectj/weaver/ResolvedTypeMunger.java
index ec4b76b2d..55e0a4e3e 100644
--- a/weaver/src/org/aspectj/weaver/ResolvedTypeMunger.java
+++ b/weaver/src/org/aspectj/weaver/ResolvedTypeMunger.java
@@ -54,7 +54,7 @@ public abstract class ResolvedTypeMunger {
     	//System.err.println("matching: " + this + " to " + matchType + " onType = " + onType);
    		if (matchType.equals(onType)) { 
    			if (!onType.isExposedToWeaver()) {
-   				if (onType.getWeaverState() != null) {
+   				if (onType.getWeaverState() == null) {
 	   				if (matchType.getWorld().getLint().typeNotExposedToWeaver.isEnabled()) {
 	   					matchType.getWorld().getLint().typeNotExposedToWeaver.signal(
 	   						matchType.getName(), signature.getSourceLocation());