Move @SuppressWarnings("unused") up to the top level of JavaASTGenerationVisitor, since all the invocation is reflective.

Review by: spoon


git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@7475 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java b/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java
index 14845b9..156418b 100644
--- a/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java
+++ b/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java
@@ -224,6 +224,8 @@
    * have to optimize out those dead blocks early and never try to translate
    * them to our AST.
    */
+  // Reflective invocation causes unused warnings.
+  @SuppressWarnings("unused")
   private static class JavaASTGenerationVisitor {
     private static InternalCompilerException translateException(JNode node,
         Throwable e) {
@@ -1143,7 +1145,6 @@
       return call;
     }
 
-    @SuppressWarnings("unused")
     JExpression processExpression(NullLiteral x) {
       return program.getLiteralNull();
     }
@@ -1550,7 +1551,6 @@
       return stmt;
     }
 
-    @SuppressWarnings("unused")
     JStatement processStatement(EmptyStatement x) {
       return null;
     }
@@ -1806,7 +1806,6 @@
           finallyBlock);
     }
 
-    @SuppressWarnings("unused")
     JStatement processStatement(TypeDeclaration x) {
       // do nothing -- the local class is treated at the program level
       return null;
@@ -1846,7 +1845,6 @@
       return jstatements;
     }
 
-    @SuppressWarnings("unused")
     JMethodCall processSuperConstructorCall(ExplicitConstructorCall x) {
       SourceInfo info = makeSourceInfo(x);
       JMethod ctor = (JMethod) typeMap.get(x.binding);