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/07/21 15:30:42 UTC

svn commit: r1364083 - /lucene/dev/branches/pforcodec_3892/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextFieldsReader.java

Author: mikemccand
Date: Sat Jul 21 13:30:42 2012
New Revision: 1364083

URL: http://svn.apache.org/viewvc?rev=1364083&view=rev
Log:
LUCENE-3892: fix SimpleText reuse bug w/ start/endOffset

Modified:
    lucene/dev/branches/pforcodec_3892/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextFieldsReader.java

Modified: lucene/dev/branches/pforcodec_3892/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextFieldsReader.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/pforcodec_3892/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextFieldsReader.java?rev=1364083&r1=1364082&r2=1364083&view=diff
==============================================================================
--- lucene/dev/branches/pforcodec_3892/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextFieldsReader.java (original)
+++ lucene/dev/branches/pforcodec_3892/lucene/core/src/java/org/apache/lucene/codecs/simpletext/SimpleTextFieldsReader.java Sat Jul 21 13:30:42 2012
@@ -343,8 +343,8 @@ class SimpleTextFieldsReader extends Fie
     private long nextDocStart;
     private boolean readOffsets;
     private boolean readPositions;
-    private int startOffset = -1;
-    private int endOffset = -1;
+    private int startOffset;
+    private int endOffset;
 
     public SimpleTextDocsAndPositionsEnum() {
       this.inStart = SimpleTextFieldsReader.this.in;
@@ -361,6 +361,10 @@ class SimpleTextFieldsReader extends Fie
       docID = -1;
       readPositions = indexOptions.compareTo(IndexOptions.DOCS_AND_FREQS_AND_POSITIONS) >= 0;
       readOffsets = indexOptions.compareTo(IndexOptions.DOCS_AND_FREQS_AND_POSITIONS_AND_OFFSETS) >= 0;
+      if (!readOffsets) {
+        startOffset = -1;
+        endOffset = -1;
+      }
       return this;
     }