Author: dbrosius
Date: Sun Oct 18 02:00:16 2009
New Revision: 826340

URL: http://svn.apache.org/viewvc?rev=826340&view=rev
Log:
code assumes Utility.getAnnotationAttributes won't return null, but does. 
convert it to just return an Attribute[0] instead of null

Modified:
    jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java
    jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/FieldGen.java
    jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/MethodGen.java

Modified: 
jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java
URL: 
http://svn.apache.org/viewvc/jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java?rev=826340&r1=826339&r2=826340&view=diff
==============================================================================
--- jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java 
(original)
+++ jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java Sun 
Oct 18 02:00:16 2009
@@ -1356,7 +1356,7 @@
      */
     public static Attribute[] getAnnotationAttributes(ConstantPoolGen cp,List 
vec) {
        
-       if (vec.isEmpty()) return null;
+       if (vec.isEmpty()) return new Attribute[0];
        
        try {
                int countVisible   = 0;

Modified: jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/FieldGen.java
URL: 
http://svn.apache.org/viewvc/jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/FieldGen.java?rev=826340&r1=826339&r2=826340&view=diff
==============================================================================
--- jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/FieldGen.java 
(original)
+++ jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/FieldGen.java Sun 
Oct 18 02:00:16 2009
@@ -227,11 +227,9 @@
     
     private void addAnnotationsAsAttribute(ConstantPoolGen cp) {
        Attribute[] attrs = Utility.getAnnotationAttributes(cp,annotation_vec);
-       if (attrs!=null) {
-          for (int i = 0; i < attrs.length; i++) {
-                 addAttribute(attrs[i]);
-         }
-       }
+        for (int i = 0; i < attrs.length; i++) {
+               addAttribute(attrs[i]);
+       }
       }
 
 

Modified: 
jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/MethodGen.java
URL: 
http://svn.apache.org/viewvc/jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/MethodGen.java?rev=826340&r1=826339&r2=826340&view=diff
==============================================================================
--- jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/MethodGen.java 
(original)
+++ jakarta/bcel/trunk/src/main/java/org/apache/bcel/generic/MethodGen.java Sun 
Oct 18 02:00:16 2009
@@ -634,11 +634,9 @@
     
     public void addAnnotationsAsAttribute(ConstantPoolGen cp) {
        Attribute[] attrs = Utility.getAnnotationAttributes(cp,annotation_vec);
-       if (attrs!=null) {
-          for (int i = 0; i < attrs.length; i++) {
-                 addAttribute(attrs[i]);
-         }
-       }
+        for (int i = 0; i < attrs.length; i++) {
+               addAttribute(attrs[i]);
+       }
       }
       
       public void addParameterAnnotationsAsAttribute(ConstantPoolGen cp) {



---------------------------------------------------------------------
To unsubscribe, e-mail: bcel-dev-unsubscr...@jakarta.apache.org
For additional commands, e-mail: bcel-dev-h...@jakarta.apache.org

Reply via email to