266420: rogue unused private warning

diff --git a/tests/src/org/aspectj/systemtest/ajc164/Ajc164Tests.java b/tests/src/org/aspectj/systemtest/ajc164/Ajc164Tests.java
index 74175ea6a8be3441b3a9a558fa523da86b176821..d2971ab9e74de3a1bc66bd96f05a4afbc334e2d9 100644 (file)
--- 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 b7cba242444c0ad45e645b3d05184415b2b23adf..12b61f24086d672d597067881c5592af552c8442 100644 (file)
--- 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"/>
     <!--