You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by ot...@apache.org on 2009/01/14 06:34:07 UTC

svn commit: r734336 - /lucene/solr/trunk/src/java/org/apache/solr/highlight/GapFragmenter.java

Author: otis
Date: Tue Jan 13 21:34:06 2009
New Revision: 734336

URL: http://svn.apache.org/viewvc?rev=734336&view=rev
Log:
SOLR-953 - Small code simplification

Modified:
    lucene/solr/trunk/src/java/org/apache/solr/highlight/GapFragmenter.java

Modified: lucene/solr/trunk/src/java/org/apache/solr/highlight/GapFragmenter.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/highlight/GapFragmenter.java?rev=734336&r1=734335&r2=734336&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/highlight/GapFragmenter.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/highlight/GapFragmenter.java Tue Jan 13 21:34:06 2009
@@ -75,7 +75,7 @@
    * the gap as a fragment delimiter.
    */
   public static final int INCREMENT_THRESHOLD = 50;
-  protected int fragOffsetAccum = 0;
+  protected int fragOffset = 0;
   
   public LuceneGapFragmenter() {
   }
@@ -88,7 +88,7 @@
    * @see org.apache.lucene.search.highlight.TextFragmenter#start(java.lang.String)
    */
   public void start(String originalText) {
-    fragOffsetAccum = 0;
+    fragOffset = 0;
   }
 
   /* (non-Javadoc)
@@ -96,10 +96,10 @@
    */
   public boolean isNewFragment(Token token) {
     boolean isNewFrag = 
-      token.endOffset() >= fragOffsetAccum + getFragmentSize() ||
+      token.endOffset() >= fragOffset + getFragmentSize() ||
       token.getPositionIncrement() > INCREMENT_THRESHOLD;
     if(isNewFrag) {
-        fragOffsetAccum += token.endOffset() - fragOffsetAccum;
+        fragOffset = token.endOffset();
     }
     return isNewFrag;
   }