You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ba...@apache.org on 2010/10/10 20:34:48 UTC
svn commit: r1006333 -
/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/AnnotationUtils.java
Author: bayard
Date: Sun Oct 10 18:34:48 2010
New Revision: 1006333
URL: http://svn.apache.org/viewvc?rev=1006333&view=rev
Log:
Changing the StringBuilder to build around an empty String and not null, as null will cause an NPE. Reported by FindBugs
Modified:
commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/AnnotationUtils.java
Modified: commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/AnnotationUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/AnnotationUtils.java?rev=1006333&r1=1006332&r2=1006333&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/AnnotationUtils.java (original)
+++ commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/AnnotationUtils.java Sun Oct 10 18:34:48 2010
@@ -59,7 +59,7 @@ public class AnnotationUtils {
break;
}
}
- return new StringBuilder(annotationType == null ? null : annotationType.getName())
+ return new StringBuilder(annotationType == null ? "" : annotationType.getName())
.insert(0, '@').toString();
}