[jboss-cvs] JBossAS SVN: r92285 - in projects/annotations/trunk/core/src/main/java/org/jboss/annotations: javalangreflect and 1 other directories.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Wed Aug 12 17:13:10 EDT 2009


Author: jesper.pedersen
Date: 2009-08-12 17:13:09 -0400 (Wed, 12 Aug 2009)
New Revision: 92285

Modified:
   projects/annotations/trunk/core/src/main/java/org/jboss/annotations/impl/AbstractAnnotationScanner.java
   projects/annotations/trunk/core/src/main/java/org/jboss/annotations/impl/AbstractJavassistAnnotationScanner.java
   projects/annotations/trunk/core/src/main/java/org/jboss/annotations/javalangreflect/JavaClass.java
   projects/annotations/trunk/core/src/main/java/org/jboss/annotations/javassistclasspool/JavassistClassPool.java
Log:
QA fixes

Modified: projects/annotations/trunk/core/src/main/java/org/jboss/annotations/impl/AbstractAnnotationScanner.java
===================================================================
--- projects/annotations/trunk/core/src/main/java/org/jboss/annotations/impl/AbstractAnnotationScanner.java	2009-08-12 21:05:20 UTC (rev 92284)
+++ projects/annotations/trunk/core/src/main/java/org/jboss/annotations/impl/AbstractAnnotationScanner.java	2009-08-12 21:13:09 UTC (rev 92285)
@@ -24,9 +24,6 @@
 
 import org.jboss.annotations.AnnotationRepository;
 import org.jboss.annotations.AnnotationScanner;
-import org.jboss.annotations.impl.BinaryLoader;
-import org.jboss.annotations.impl.Scan;
-import org.jboss.annotations.impl.XMLLoader;
 import org.jboss.annotations.util.ClassScanner;
 import org.jboss.annotations.util.JarScanner;
 
@@ -47,8 +44,8 @@
  */
 public abstract class AbstractAnnotationScanner implements AnnotationScanner
 {
-   private static Logger log;
-   private static boolean trace; 
+   private Logger log;
+   private boolean trace; 
 
    /** Name of the binary metadata */
    public static final String JBOSS_ANNOTATION_BINARY = "jboss-annotation.ser";
@@ -158,7 +155,8 @@
       }
       catch (Throwable t)
       {
-         log.finest(t.getMessage());
+         if (trace)
+            log.finest(t.getMessage());
       }
       finally
       {

Modified: projects/annotations/trunk/core/src/main/java/org/jboss/annotations/impl/AbstractJavassistAnnotationScanner.java
===================================================================
--- projects/annotations/trunk/core/src/main/java/org/jboss/annotations/impl/AbstractJavassistAnnotationScanner.java	2009-08-12 21:05:20 UTC (rev 92284)
+++ projects/annotations/trunk/core/src/main/java/org/jboss/annotations/impl/AbstractJavassistAnnotationScanner.java	2009-08-12 21:13:09 UTC (rev 92285)
@@ -24,7 +24,6 @@
 
 import org.jboss.annotations.Annotation;
 import org.jboss.annotations.AnnotationType;
-import org.jboss.annotations.impl.ScanClass;
 
 import java.util.Collection;
 import java.util.HashSet;
@@ -44,8 +43,8 @@
  */
 public abstract class AbstractJavassistAnnotationScanner extends AbstractAnnotationScanner
 {
-   private static Logger log;
-   private static boolean trace; 
+   private Logger log;
+   private boolean trace; 
 
    /**
     * Constructor

Modified: projects/annotations/trunk/core/src/main/java/org/jboss/annotations/javalangreflect/JavaClass.java
===================================================================
--- projects/annotations/trunk/core/src/main/java/org/jboss/annotations/javalangreflect/JavaClass.java	2009-08-12 21:05:20 UTC (rev 92284)
+++ projects/annotations/trunk/core/src/main/java/org/jboss/annotations/javalangreflect/JavaClass.java	2009-08-12 21:13:09 UTC (rev 92285)
@@ -31,8 +31,8 @@
 import org.jboss.annotations.impl.ScanClass;
 
 import java.io.Closeable;
+import java.io.File;
 import java.io.IOException;
-import java.io.File;
 import java.lang.reflect.Constructor;
 import java.lang.reflect.Field;
 import java.lang.reflect.Method;
@@ -192,7 +192,9 @@
                                  
                                  if (scanClass == null || scanClass.hasMethod(method.getName(), parameterTypes))
                                  {
-                                    java.lang.annotation.Annotation[] methodAnnotations = method.getDeclaredAnnotations();
+                                    java.lang.annotation.Annotation[] methodAnnotations = 
+                                       method.getDeclaredAnnotations();
+
                                     if (methodAnnotations != null)
                                     {
                                        for (java.lang.annotation.Annotation annotation : methodAnnotations)
@@ -218,7 +220,9 @@
                            
                                     if (scanClass == null || scanClass.hasField(field.getName()))
                                     {
-                                       java.lang.annotation.Annotation[] fieldAnnotations = field.getDeclaredAnnotations();
+                                       java.lang.annotation.Annotation[] fieldAnnotations = 
+                                          field.getDeclaredAnnotations();
+
                                        if (fieldAnnotations != null)
                                        {
                                           for (java.lang.annotation.Annotation annotation : fieldAnnotations)
@@ -240,7 +244,9 @@
                               {
                                  if (!interfaceClass.getName().startsWith("java."))
                                  {
-                                    java.lang.annotation.Annotation[] interfaceAnnotations = interfaceClass.getAnnotations();
+                                    java.lang.annotation.Annotation[] interfaceAnnotations = 
+                                       interfaceClass.getAnnotations();
+
                                     if (interfaceAnnotations != null)
                                     {
                                        for (java.lang.annotation.Annotation annotation : interfaceAnnotations)
@@ -260,7 +266,9 @@
                                           if (trace)
                                              log.finest("Interface Method=" + method.getName());
 
-                                          java.lang.annotation.Annotation[] methodAnnotations = method.getDeclaredAnnotations();
+                                          java.lang.annotation.Annotation[] methodAnnotations = 
+                                             method.getDeclaredAnnotations();
+
                                           if (methodAnnotations != null)
                                           {
                                              String[] parameterTypes = null;

Modified: projects/annotations/trunk/core/src/main/java/org/jboss/annotations/javassistclasspool/JavassistClassPool.java
===================================================================
--- projects/annotations/trunk/core/src/main/java/org/jboss/annotations/javassistclasspool/JavassistClassPool.java	2009-08-12 21:05:20 UTC (rev 92284)
+++ projects/annotations/trunk/core/src/main/java/org/jboss/annotations/javassistclasspool/JavassistClassPool.java	2009-08-12 21:13:09 UTC (rev 92285)
@@ -30,8 +30,8 @@
 import org.jboss.annotations.impl.ScanClass;
 
 import java.io.Closeable;
+import java.io.File;
 import java.io.IOException;
-import java.io.File;
 import java.net.URL;
 import java.net.URLClassLoader;
 import java.util.Arrays;




More information about the jboss-cvs-commits mailing list