You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ds...@apache.org on 2013/04/07 09:03:46 UTC

svn commit: r1465330 - in /lucene/dev/trunk/lucene: CHANGES.txt spatial/src/java/org/apache/lucene/spatial/prefix/tree/Cell.java

Author: dsmiley
Date: Sun Apr  7 07:03:46 2013
New Revision: 1465330

URL: http://svn.apache.org/r1465330
Log:
LUCENE-4914: SpatialPrefixTree Cell.reset() forgot the 'leaf' flag.

Modified:
    lucene/dev/trunk/lucene/CHANGES.txt
    lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/prefix/tree/Cell.java

Modified: lucene/dev/trunk/lucene/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/CHANGES.txt?rev=1465330&r1=1465329&r2=1465330&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/CHANGES.txt (original)
+++ lucene/dev/trunk/lucene/CHANGES.txt Sun Apr  7 07:03:46 2013
@@ -225,6 +225,10 @@ Bug Fixes
 * LUCENE-4877: Throw exception for invalid arguments in analysis factories.
   (Steve Rowe, Uwe Schindler, Robert Muir)
 
+* LUCENE-4914: SpatialPrefixTree's Node/Cell.reset() forgot to reset the 'leaf'
+  flag.  It affects SpatialRecursivePrefixTreeStrategy on non-point indexed
+  shapes, as of Lucene 4.2. (David Smiley)
+
 Documentation
 
 * LUCENE-4841: Added example SimpleSortedSetFacetsExample to show how

Modified: lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/prefix/tree/Cell.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/prefix/tree/Cell.java?rev=1465330&r1=1465329&r2=1465330&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/prefix/tree/Cell.java (original)
+++ lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/prefix/tree/Cell.java Sun Apr  7 07:03:46 2013
@@ -91,6 +91,8 @@ public abstract class Cell implements Co
     if (bytes[b_off + b_len - 1] == LEAF_BYTE) {
       b_len--;
       setLeaf();
+    } else {
+      leaf = false;
     }
   }