You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sh...@apache.org on 2011/01/16 15:14:55 UTC
svn commit: r1059566 -
/lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/search/FieldValueHitQueue.java
Author: shaie
Date: Sun Jan 16 14:14:55 2011
New Revision: 1059566
URL: http://svn.apache.org/viewvc?rev=1059566&view=rev
Log:
OneComparatorFieldValueHitQueue redundantly checks array size - it is alredy guaranteed to be of the right one by FVHQ.create()
Modified:
lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/search/FieldValueHitQueue.java
Modified: lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/search/FieldValueHitQueue.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/search/FieldValueHitQueue.java?rev=1059566&r1=1059565&r2=1059566&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/search/FieldValueHitQueue.java (original)
+++ lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/search/FieldValueHitQueue.java Sun Jan 16 14:14:55 2011
@@ -59,9 +59,6 @@ public abstract class FieldValueHitQueue
public OneComparatorFieldValueHitQueue(SortField[] fields, int size)
throws IOException {
super(fields);
- if (fields.length == 0) {
- throw new IllegalArgumentException("Sort must contain at least one field");
- }
SortField field = fields[0];
comparator = field.getComparator(size, 0);