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

toad at freenetproject.org toad at freenetproject.org
Sat Feb 10 17:56:08 UTC 2007


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);
     }




More information about the cvs mailing list