accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From medi...@apache.org
Subject svn commit: r1384021 - /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/Value.java
Date Wed, 12 Sep 2012 16:24:55 GMT
Author: medined
Date: Wed Sep 12 16:24:54 2012
New Revision: 1384021

URL: http://svn.apache.org/viewvc?rev=1384021&view=rev
Log:
toString should not return null according to FindBugs

Modified:
    accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/Value.java

Modified: accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/Value.java
URL: http://svn.apache.org/viewvc/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/Value.java?rev=1384021&r1=1384020&r2=1384021&view=diff
==============================================================================
--- accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/Value.java (original)
+++ accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/Value.java Wed Sep 12
16:24:54 2012
@@ -199,12 +199,13 @@ public class Value implements WritableCo
   
   @Override
   public String toString() {
+	String retValue = "";
     try {
-      return new String(get(), Constants.VALUE_ENCODING);
+      retValue = new String(get(), Constants.VALUE_ENCODING);
     } catch (UnsupportedEncodingException e) {
       log.error(e.toString());
-      return null;
     }
+    return retValue;
   }
   
   /**



Mime
View raw message