You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by rm...@apache.org on 2011/02/02 21:33:03 UTC
svn commit: r1066618 -
/lucene/dev/branches/bulkpostings/lucene/src/test/org/apache/lucene/index/codecs/pfordelta2/TestPForDelta2.java
Author: rmuir
Date: Wed Feb 2 20:33:03 2011
New Revision: 1066618
URL: http://svn.apache.org/viewvc?rev=1066618&view=rev
Log:
fix sneaky off-by-one: the test data could lie on a block boundary
Modified:
lucene/dev/branches/bulkpostings/lucene/src/test/org/apache/lucene/index/codecs/pfordelta2/TestPForDelta2.java
Modified: lucene/dev/branches/bulkpostings/lucene/src/test/org/apache/lucene/index/codecs/pfordelta2/TestPForDelta2.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/bulkpostings/lucene/src/test/org/apache/lucene/index/codecs/pfordelta2/TestPForDelta2.java?rev=1066618&r1=1066617&r2=1066618&view=diff
==============================================================================
--- lucene/dev/branches/bulkpostings/lucene/src/test/org/apache/lucene/index/codecs/pfordelta2/TestPForDelta2.java (original)
+++ lucene/dev/branches/bulkpostings/lucene/src/test/org/apache/lucene/index/codecs/pfordelta2/TestPForDelta2.java Wed Feb 2 20:33:03 2011
@@ -63,14 +63,14 @@ public class TestPForDelta2 extends Luce
assertTrue(pointerMax > 0);
for(int i=0;i<testDataSize;i++) {
- final int expected = testData[i];
- final int actual = buffer[pointer++];
- assertEquals(actual + " != " + expected, expected, actual);
if (pointer == pointerMax) {
pointerMax = r.fill();
assertTrue(pointerMax > 0);
pointer = 0;
}
+ final int expected = testData[i];
+ final int actual = buffer[pointer++];
+ assertEquals(actual + " != " + expected, expected, actual);
}
in.close();
dir.close();