266420: rogue unused private warning

2 files changed, 36 insertions, 12 deletions

diff --git a/tests/src/org/aspectj/systemtest/ajc164/Ajc164Tests.java b/tests/src/org/aspectj/systemtest/ajc164/Ajc164Tests.java
index 74175ea6a..d2971ab9e 100644
--- a/tests/src/org/aspectj/systemtest/ajc164/Ajc164Tests.java
+++ b/tests/src/org/aspectj/systemtest/ajc164/Ajc164Tests.java
@@ -34,13 +34,23 @@ import org.aspectj.testing.XMLBasedAjcTestCase;
 
 public class Ajc164Tests extends org.aspectj.testing.XMLBasedAjcTestCase {
 
+	public void testUnusedPrivateWarning_pr266420() {
+		runTest("unused private warning");
+	}
+
+	public void testUnusedPrivateWarning_pr266420_2() {
+		runTest("unused private warning - 2");
+	}
+
+	public void testUnusedPrivateWarning_pr266420_3() {
+		runTest("unused private warning - 3");
+	}
+
 	/**
-	 * This test program can be used to compare handles for faulted in binary
-	 * aspects with handles that would be used if the aspect was available as
-	 * source. There are two compile steps in the xml for the test - commenting
-	 * out the first will allow the source handles to be seen, leaving it in
-	 * will switch to binary. Effectively the only difference should be that in
-	 * the binary case the handles are prefixed 'binaries'.
+	 * This test program can be used to compare handles for faulted in binary aspects with handles that would be used if the aspect
+	 * was available as source. There are two compile steps in the xml for the test - commenting out the first will allow the source
+	 * handles to be seen, leaving it in will switch to binary. Effectively the only difference should be that in the binary case
+	 * the handles are prefixed 'binaries'.
 	 */
 	public void testItdsAspectPathModel_pr265729_1() {
 		runTest("aspectpath model");
@@ -242,11 +252,9 @@ public class Ajc164Tests extends org.aspectj.testing.XMLBasedAjcTestCase {
 	}
 
 	/**
-	 * This test checks that local variable table for the interMethodDispatcher
-	 * is built correctly, for the related code see
-	 * IntertypeMethodDeclaration.generateDispatchMethod(). It checks non-static
-	 * and static ITDs. Once the information here is correct then around advice
-	 * on ITDs can also be correct.
+	 * This test checks that local variable table for the interMethodDispatcher is built correctly, for the related code see
+	 * IntertypeMethodDeclaration.generateDispatchMethod(). It checks non-static and static ITDs. Once the information here is
+	 * correct then around advice on ITDs can also be correct.
 	 */
 	public void testBrokenLVT_pr194314_3() throws Exception {
 		runTest("broken lvt - 3");
diff --git a/tests/src/org/aspectj/systemtest/ajc164/ajc164.xml b/tests/src/org/aspectj/systemtest/ajc164/ajc164.xml
index b7cba2424..12b61f240 100644
--- a/tests/src/org/aspectj/systemtest/ajc164/ajc164.xml
+++ b/tests/src/org/aspectj/systemtest/ajc164/ajc164.xml
@@ -2,7 +2,23 @@
 
 <suite>
    
-   <ajc-test dir="bugs164/pr265729" title="aspectpath model">
+   <ajc-test dir="bugs164/pr266420" title="unused private warning">
+     <compile files="A.java" options="-1.5 -warn:unusedPrivate"/>
+   </ajc-test>
+   
+   <ajc-test dir="bugs164/pr266420" title="unused private warning - 2">
+     <compile files="B.java" options="-1.5 -warn:unusedPrivate">
+       <message kind="warning" text="The field B.i is never read locally"/>
+     </compile>
+   </ajc-test>
+   
+   <ajc-test dir="bugs164/pr266420" title="unused private warning - 3">
+     <compile files="C.java" options="-1.5 -warn:unusedPrivate">
+       <message kind="warning" text="The field C.j is never read locally"/>
+     </compile>
+   </ajc-test>
+   
+    <ajc-test dir="bugs164/pr265729" title="aspectpath model">
     <compile files="Aspect.java Orange.java Strawberry.java Fruit.java" outjar="library.jar" options="-emacssym"/>
     <compile files="Orange.java Strawberry.java Fruit.java" aspectpath="library.jar" options="-emacssym"/>
     <!--