You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bcel-dev@jakarta.apache.org by db...@apache.org on 2009/10/18 04:00:17 UTC
svn commit: r826340 - in /jakarta/bcel/trunk/src/main/java/org/apache/bcel:
classfile/Utility.java generic/FieldGen.java generic/MethodGen.java
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-unsubscribe@jakarta.apache.org
For additional commands, e-mail: bcel-dev-help@jakarta.apache.org