You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@abdera.apache.org by jm...@apache.org on 2009/05/06 23:57:46 UTC

svn commit: r772439 - /abdera/java/trunk/dependencies/i18n/src/main/java/org/apache/abdera/i18n/text/UrlEncoding.java

Author: jmsnell
Date: Wed May  6 21:57:46 2009
New Revision: 772439

URL: http://svn.apache.org/viewvc?rev=772439&view=rev
Log:
Fix NPE waiting to happen

Modified:
    abdera/java/trunk/dependencies/i18n/src/main/java/org/apache/abdera/i18n/text/UrlEncoding.java

Modified: abdera/java/trunk/dependencies/i18n/src/main/java/org/apache/abdera/i18n/text/UrlEncoding.java
URL: http://svn.apache.org/viewvc/abdera/java/trunk/dependencies/i18n/src/main/java/org/apache/abdera/i18n/text/UrlEncoding.java?rev=772439&r1=772438&r2=772439&view=diff
==============================================================================
--- abdera/java/trunk/dependencies/i18n/src/main/java/org/apache/abdera/i18n/text/UrlEncoding.java (original)
+++ abdera/java/trunk/dependencies/i18n/src/main/java/org/apache/abdera/i18n/text/UrlEncoding.java Wed May  6 21:57:46 2009
@@ -367,7 +367,7 @@
     String enc, 
     Filter... filters) 
       throws UnsupportedEncodingException {
-    if (s == null) return s.toString();
+    if (s == null) return null;
     StringBuilder sb = new StringBuilder();
 
     for (int n = 0; n < s.length(); n++) {