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;