You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by gs...@apache.org on 2010/12/02 18:52:05 UTC
svn commit: r1041497 - in /lucene/dev/branches/branch_3x: ./ lucene/ solr/
solr/src/java/org/apache/solr/search/FunctionQParser.java
Author: gsingers
Date: Thu Dec 2 17:52:05 2010
New Revision: 1041497
URL: http://svn.apache.org/viewvc?rev=1041497&view=rev
Log:
fix parsing of empty value source list [from revision 1002002]
Modified:
lucene/dev/branches/branch_3x/ (props changed)
lucene/dev/branches/branch_3x/lucene/ (props changed)
lucene/dev/branches/branch_3x/solr/ (props changed)
lucene/dev/branches/branch_3x/solr/src/java/org/apache/solr/search/FunctionQParser.java
Modified: lucene/dev/branches/branch_3x/solr/src/java/org/apache/solr/search/FunctionQParser.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/solr/src/java/org/apache/solr/search/FunctionQParser.java?rev=1041497&r1=1041496&r2=1041497&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/solr/src/java/org/apache/solr/search/FunctionQParser.java (original)
+++ lucene/dev/branches/branch_3x/solr/src/java/org/apache/solr/search/FunctionQParser.java Thu Dec 2 17:52:05 2010
@@ -214,9 +214,8 @@ public class FunctionQParser extends QPa
*/
public List<ValueSource> parseValueSourceList() throws ParseException {
List<ValueSource> sources = new ArrayList<ValueSource>(3);
- for (;;) {
- sources.add(parseValueSource(false));
- if (! consumeArgumentDelimiter()) break;
+ while (hasMoreArguments()) {
+ sources.add(parseValueSource(true));
}
return sources;
}