You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ds...@apache.org on 2012/08/19 23:42:02 UTC
svn commit: r1374857 - in
/lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query:
SpatialArgs.java SpatialArgsParser.java
Author: dsmiley
Date: Sun Aug 19 21:42:02 2012
New Revision: 1374857
URL: http://svn.apache.org/viewvc?rev=1374857&view=rev
Log:
LUCENE-4197 SpatialArgs- remove unused min and max params
Modified:
lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgs.java
lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgsParser.java
Modified: lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgs.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgs.java?rev=1374857&r1=1374856&r2=1374857&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgs.java (original)
+++ lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgs.java Sun Aug 19 21:42:02 2012
@@ -35,10 +35,6 @@ public class SpatialArgs {
private Shape shape;
private double distPrecision = DEFAULT_DIST_PRECISION;
- // Useful for 'distance' calculations
- private Double min;
- private Double max;
-
public SpatialArgs(SpatialOperation operation) {
this.operation = operation;
}
@@ -60,12 +56,6 @@ public class SpatialArgs {
StringBuilder str = new StringBuilder();
str.append(operation.getName()).append('(');
str.append(shape.toString());
- if (min != null) {
- str.append(" min=").append(min);
- }
- if (max != null) {
- str.append(" max=").append(max);
- }
str.append(" distPrec=").append(String.format(Locale.ROOT, "%.2f%%", distPrecision / 100d));
str.append(')');
return str.toString();
@@ -111,19 +101,4 @@ public class SpatialArgs {
this.distPrecision = distPrecision;
}
- public Double getMin() {
- return min;
- }
-
- public void setMin(Double min) {
- this.min = min;
- }
-
- public Double getMax() {
- return max;
- }
-
- public void setMax(Double max) {
- this.max = max;
- }
}
Modified: lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgsParser.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgsParser.java?rev=1374857&r1=1374856&r2=1374857&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgsParser.java (original)
+++ lucene/dev/trunk/lucene/spatial/src/java/org/apache/lucene/spatial/query/SpatialArgsParser.java Sun Aug 19 21:42:02 2012
@@ -75,8 +75,6 @@ public class SpatialArgsParser {
body = v.substring(edx + 1).trim();
if (body.length() > 0) {
Map<String, String> aa = parseMap(body);
- args.setMin(readDouble(aa.remove("min")));
- args.setMax(readDouble(aa.remove("max")));
args.setDistPrecision(readDouble(aa.remove("distPrec")));
if (!aa.isEmpty()) {
throw new InvalidSpatialArgument("unused parameters: " + aa, null);