You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@velocity.apache.org by nb...@apache.org on 2007/08/08 19:53:37 UTC
svn commit: r563963 -
/velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/generic/SortTool.java
Author: nbubna
Date: Wed Aug 8 10:53:35 2007
New Revision: 563963
URL: http://svn.apache.org/viewvc?view=rev&rev=563963
Log:
fix NPE (found by FindBugs)
Modified:
velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/generic/SortTool.java
Modified: velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/generic/SortTool.java
URL: http://svn.apache.org/viewvc/velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/generic/SortTool.java?view=diff&rev=563963&r1=563962&r2=563963
==============================================================================
--- velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/generic/SortTool.java (original)
+++ velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/generic/SortTool.java Wed Aug 8 10:53:35 2007
@@ -279,7 +279,7 @@
//TODO: make it optional whether or not case is ignored
comparison = ((String)left).compareToIgnoreCase((String)right);
}
- else
+ else if (left != null)
{
comparison = left.compareTo(right);
}