Return-Path: Delivered-To: apmail-lucene-java-commits-archive@www.apache.org Received: (qmail 696 invoked from network); 29 Apr 2008 21:51:29 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 29 Apr 2008 21:51:29 -0000 Received: (qmail 56580 invoked by uid 500); 29 Apr 2008 21:51:31 -0000 Delivered-To: apmail-lucene-java-commits-archive@lucene.apache.org Received: (qmail 56555 invoked by uid 500); 29 Apr 2008 21:51:31 -0000 Mailing-List: contact java-commits-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: java-dev@lucene.apache.org Delivered-To: mailing list java-commits@lucene.apache.org Received: (qmail 56544 invoked by uid 99); 29 Apr 2008 21:51:31 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 29 Apr 2008 14:51:31 -0700 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 29 Apr 2008 21:50:44 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id 7399723889F5; Tue, 29 Apr 2008 14:51:03 -0700 (PDT) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r652162 - in /lucene/java/branches/lucene_2_3/src: java/org/apache/lucene/index/DocumentsWriter.java test/org/apache/lucene/index/TestIndexWriter.java Date: Tue, 29 Apr 2008 21:51:03 -0000 To: java-commits@lucene.apache.org From: mikemccand@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20080429215103.7399723889F5@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: mikemccand Date: Tue Apr 29 14:51:03 2008 New Revision: 652162 URL: http://svn.apache.org/viewvc?rev=652162&view=rev Log: LUCENE-1255: revert this change (allow pos=-1 again) since it's not backwards compatible Modified: lucene/java/branches/lucene_2_3/src/java/org/apache/lucene/index/DocumentsWriter.java lucene/java/branches/lucene_2_3/src/test/org/apache/lucene/index/TestIndexWriter.java Modified: lucene/java/branches/lucene_2_3/src/java/org/apache/lucene/index/DocumentsWriter.java URL: http://svn.apache.org/viewvc/lucene/java/branches/lucene_2_3/src/java/org/apache/lucene/index/DocumentsWriter.java?rev=652162&r1=652161&r2=652162&view=diff ============================================================================== --- lucene/java/branches/lucene_2_3/src/java/org/apache/lucene/index/DocumentsWriter.java (original) +++ lucene/java/branches/lucene_2_3/src/java/org/apache/lucene/index/DocumentsWriter.java Tue Apr 29 14:51:03 2008 @@ -1522,9 +1522,6 @@ Token token = stream.next(localToken); if (token == null) break; position += (token.getPositionIncrement() - 1); - // LUCENE-1255: don't allow negative positon - if (position < 0) - position = 0; addPosition(token); if (++length >= maxFieldLength) { if (infoStream != null) Modified: lucene/java/branches/lucene_2_3/src/test/org/apache/lucene/index/TestIndexWriter.java URL: http://svn.apache.org/viewvc/lucene/java/branches/lucene_2_3/src/test/org/apache/lucene/index/TestIndexWriter.java?rev=652162&r1=652161&r2=652162&view=diff ============================================================================== --- lucene/java/branches/lucene_2_3/src/test/org/apache/lucene/index/TestIndexWriter.java (original) +++ lucene/java/branches/lucene_2_3/src/test/org/apache/lucene/index/TestIndexWriter.java Tue Apr 29 14:51:03 2008 @@ -3029,7 +3029,7 @@ TermPositions tps = s.getIndexReader().termPositions(new Term("field", "a")); assertTrue(tps.next()); assertEquals(1, tps.freq()); - assertEquals(0, tps.nextPosition()); + assertEquals(-1, tps.nextPosition()); assertTrue(_TestUtil.checkIndex(dir)); s.close();