You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by us...@apache.org on 2011/12/10 19:06:58 UTC

svn commit: r1212831 - /lucene/dev/trunk/lucene/src/java/org/apache/lucene/util/ReaderUtil.java

Author: uschindler
Date: Sat Dec 10 18:06:57 2011
New Revision: 1212831

URL: http://svn.apache.org/viewvc?rev=1212831&view=rev
Log:
Fix hidden bug in ReaderContextBuilder, as we never checked the context of a MultiReader

Modified:
    lucene/dev/trunk/lucene/src/java/org/apache/lucene/util/ReaderUtil.java

Modified: lucene/dev/trunk/lucene/src/java/org/apache/lucene/util/ReaderUtil.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/src/java/org/apache/lucene/util/ReaderUtil.java?rev=1212831&r1=1212830&r2=1212831&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/src/java/org/apache/lucene/util/ReaderUtil.java (original)
+++ lucene/dev/trunk/lucene/src/java/org/apache/lucene/util/ReaderUtil.java Sat Dec 10 18:06:57 2011
@@ -189,7 +189,7 @@ public final class ReaderUtil {
         
         int newDocBase = 0;
         for (int i = 0; i < sequentialSubReaders.length; i++) {
-          build(newParent, sequentialSubReaders[i], i, newDocBase);
+          children[i] = build(newParent, sequentialSubReaders[i], i, newDocBase);
           newDocBase += sequentialSubReaders[i].maxDoc();
         }
         return newParent;