You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by gs...@apache.org on 2010/02/18 21:55:12 UTC
svn commit: r911574 -
/lucene/solr/trunk/src/java/org/apache/solr/search/function/distance/DistanceUtils.java
Author: gsingers
Date: Thu Feb 18 20:55:12 2010
New Revision: 911574
URL: http://svn.apache.org/viewvc?rev=911574&view=rev
Log:
SOLR-1779: handle all spaces between a comma properly
Modified:
lucene/solr/trunk/src/java/org/apache/solr/search/function/distance/DistanceUtils.java
Modified: lucene/solr/trunk/src/java/org/apache/solr/search/function/distance/DistanceUtils.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/search/function/distance/DistanceUtils.java?rev=911574&r1=911573&r2=911574&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/search/function/distance/DistanceUtils.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/search/function/distance/DistanceUtils.java Thu Feb 18 20:55:12 2010
@@ -139,6 +139,9 @@
for (; i < dimension; i++) {
while (start < end && externalVal.charAt(start) == ' ') start++;
while (end > start && externalVal.charAt(end - 1) == ' ') end--;
+ if (start == end){
+ break;
+ }
out[i] = externalVal.substring(start, end);
start = idx + 1;
end = externalVal.indexOf(',', start);
@@ -180,6 +183,9 @@
//TODO: abstract common code with other parsePoint
while (start < end && externalVal.charAt(start) == ' ') start++;
while (end > start && externalVal.charAt(end - 1) == ' ') end--;
+ if (start == end){
+ break;
+ }
out[i] = Double.parseDouble(externalVal.substring(start, end));
start = idx + 1;
end = externalVal.indexOf(',', start);