[freenet-cvs] r11746 - trunk/freenet/src/freenet/support

[t...@freenetproject.org]

Author: toad
Date: 2007-02-10 17:56:07 +0000 (Sat, 10 Feb 2007)
New Revision: 11746

Modified:
   trunk/freenet/src/freenet/support/SimpleFieldSet.java
Log:
Fix NPE. ,null is a valid no-op.

Modified: trunk/freenet/src/freenet/support/SimpleFieldSet.java
===================================================================
--- trunk/freenet/src/freenet/support/SimpleFieldSet.java       2007-02-10 
17:30:13 UTC (rev 11745)
+++ trunk/freenet/src/freenet/support/SimpleFieldSet.java       2007-02-10 
17:56:07 UTC (rev 11746)
@@ -201,6 +201,7 @@
      * @param value The value.
      */
     public void putSingle(String key, String value) {
+       if(value == null) return;
        if(!shortLived) value = value.intern();
        if(!put(key, value, false, false))
                throw new IllegalStateException("Value already exists: 
"+value+" but want to set "+key+" to "+value);
@@ -212,6 +213,7 @@
      * @param value The value.
      */
     public void putAppend(String key, String value) {
+       if(value == null) return;
        if(!shortLived) value = value.intern();
        put(key, value, true, false);
     }
@@ -222,6 +224,7 @@
      * @param value The value.
      */
     public void putOverwrite(String key, String value) {
+       if(value == null) return;
        if(!shortLived) value = value.intern();
        put(key, value, false, true);
     }