Author: zothar
Date: 2007-01-03 02:10:48 +0000 (Wed, 03 Jan 2007)
New Revision: 11548

Modified:
   trunk/freenet/src/freenet/support/Fields.java
Log:
Comment about an NPE I got in support/Fields.java; I'm not familiar enough with 
the code to be sure a if null test is smart there.

Modified: trunk/freenet/src/freenet/support/Fields.java
===================================================================
--- trunk/freenet/src/freenet/support/Fields.java       2007-01-03 00:07:06 UTC 
(rev 11547)
+++ trunk/freenet/src/freenet/support/Fields.java       2007-01-03 02:10:48 UTC 
(rev 11548)
@@ -157,6 +157,11 @@

        public static final String[] commaList(String ls) {
                StringTokenizer st = new StringTokenizer(ls, ",");
+               // FIXME: Next line can NPE
+               //       at java.lang.String.charAt(String.java:560)
+               //       at 
java.util.StringTokenizer.skipDelimiters(StringTokenizer.java:234)
+               //       at 
java.util.StringTokenizer.countTokens(StringTokenizer.java:406)
+               //       at freenet.support.Fields.commaList(Fields.java:160)
                String[] r = new String[st.countTokens()];
                for (int i = 0; i < r.length; i++) {
                        r[i] = st.nextToken().trim();


Reply via email to