You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2012/03/16 13:54:41 UTC

svn commit: r1301472 - in /lucene/dev/trunk/lucene/core/src/java/org/apache/lucene: codecs/simpletext/SimpleTextDocValuesConsumer.java index/MultiDocValues.java util/fst/PairOutputs.java

Author: mikemccand
Date: Fri Mar 16 12:54:40 2012
New Revision: 1301472

URL: http://svn.apache.org/viewvc?rev=1301472&view=rev
Log:
LUCENE-3877: don't print to std streams in Lucene core

Modified:
    lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextDocValuesConsumer.java
    lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/MultiDocValues.java
    lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/util/fst/PairOutputs.java

Modified: lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextDocValuesConsumer.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextDocValuesConsumer.java?rev=1301472&r1=1301471&r2=1301472&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextDocValuesConsumer.java (original)
+++ lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextDocValuesConsumer.java Fri Mar 16 12:54:40 2012
@@ -65,7 +65,6 @@ public class SimpleTextDocValuesConsumer
     hash = new BytesRefHash();
     ords = new int[0];
     this.segmentSuffix = segmentSuffix;
-
   }
 
   @Override
@@ -84,11 +83,7 @@ public class SimpleTextDocValuesConsumer
     case BYTES_VAR_SORTED:
     case BYTES_VAR_STRAIGHT:
       vSize = -1;
-      try {
       ord = hash.add(value.binaryValue());
-      } catch (NullPointerException e) {
-        System.err.println();
-      }
       break;
     case FIXED_INTS_16:
       vSize = 2;

Modified: lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/MultiDocValues.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/MultiDocValues.java?rev=1301472&r1=1301471&r2=1301472&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/MultiDocValues.java (original)
+++ lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/MultiDocValues.java Fri Mar 16 12:54:40 2012
@@ -476,9 +476,6 @@ public class MultiDocValues extends DocV
         offset =  ordToOffset[ord];
         size = (int) (ordToOffset[1 + ord] - offset);
       }
-      if (size < 0) {
-        System.out.println();
-      }
       assert size >=0;
       return data.fillSlice(bytesRef, offset, size);
      }

Modified: lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/util/fst/PairOutputs.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/util/fst/PairOutputs.java?rev=1301472&r1=1301471&r2=1301472&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/util/fst/PairOutputs.java (original)
+++ lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/util/fst/PairOutputs.java Fri Mar 16 12:54:40 2012
@@ -92,18 +92,15 @@ public class PairOutputs<A,B> extends Ou
     final boolean noOutput2 = pair.output2.equals(outputs2.getNoOutput());
 
     if (noOutput1 && pair.output1 != outputs1.getNoOutput()) {
-      System.out.println("invalid0");
       return false;
     }
 
     if (noOutput2 && pair.output2 != outputs2.getNoOutput()) {
-      System.out.println("invalid1");
       return false;
     }
 
     if (noOutput1 && noOutput2) {
       if (pair != NO_OUTPUT) {
-        System.out.println("invalid2");
         return false;
       } else {
         return true;