diff options
author | zmiao <miao.zhao@mapbox.com> | 2020-04-28 20:33:37 +0300 |
---|---|---|
committer | zmiao <miao.zhao@mapbox.com> | 2020-04-28 23:10:04 +0300 |
commit | 4a54da881ff75cc6f034176cb7f249751739e66f (patch) | |
tree | b98430ae8ed1dfd0991fffd37a6d3af6415c342f | |
parent | af14e1d510649fcd181e4da361803a5caaa5ba62 (diff) |
[core] Distance expression: Remove code and comments related to unit argument
-rw-r--r-- | src/mbgl/style/expression/distance.cpp | 16 | ||||
-rw-r--r-- | test/fixtures/expression_equality/distance.a.json | 2 | ||||
-rw-r--r-- | test/fixtures/expression_equality/distance.b.json | 2 |
3 files changed, 9 insertions, 11 deletions
diff --git a/src/mbgl/style/expression/distance.cpp b/src/mbgl/style/expression/distance.cpp index 9992ef539..5ab120634 100644 --- a/src/mbgl/style/expression/distance.cpp +++ b/src/mbgl/style/expression/distance.cpp @@ -394,28 +394,26 @@ double calculateDistance(const GeometryTileFeature& feature, optional<GeoJSON> parseValue(const style::conversion::Convertible& value, style::expression::ParsingContext& ctx) { if (isArray(value)) { - // object value, quoted with ["Distance", GeoJSONObj, "unit(optional)"] + // object value, quoted with ["Distance", GeoJSONObj] auto length = arrayLength(value); - if (length != 2 && length != 3) { - ctx.error("'distance' expression requires either one argument or two arguments, but found " + + if (length != 2) { + ctx.error("'distance' expression requires one argument, but found " + util::toString(arrayLength(value) - 1) + " instead."); return nullopt; } // Parse geometry info - const auto& argument1 = arrayMember(value, 1); - if (isObject(argument1)) { + const auto& argument = arrayMember(value, 1); + if (isObject(argument)) { style::conversion::Error error; - auto geojson = toGeoJSON(argument1, error); + auto geojson = toGeoJSON(argument, error); if (geojson && error.message.empty()) { return *geojson; } ctx.error(error.message); } } - ctx.error( - "'distance' expression needs to be an array with format [\"Distance\", GeoJSONObj, \"units\"(\"units\" is an " - "optional argument, 'meters' will be used by default)]."); + ctx.error("'distance' expression needs to be an array with format [\"Distance\", GeoJSONObj]."); return nullopt; } diff --git a/test/fixtures/expression_equality/distance.a.json b/test/fixtures/expression_equality/distance.a.json index 25079db71..5deaf2c8f 100644 --- a/test/fixtures/expression_equality/distance.a.json +++ b/test/fixtures/expression_equality/distance.a.json @@ -1,5 +1,5 @@ ["distance", { "type": "LineString", "coordinates": [[0, 0], [0, 5], [5, 5], [5, 0]] - }, "meters" + } ]
\ No newline at end of file diff --git a/test/fixtures/expression_equality/distance.b.json b/test/fixtures/expression_equality/distance.b.json index cac710cb9..4a4714354 100644 --- a/test/fixtures/expression_equality/distance.b.json +++ b/test/fixtures/expression_equality/distance.b.json @@ -1,5 +1,5 @@ ["distance", { "type": "LineString", "coordinates": [[0, 0], [0, 6], [5, 5], [5, 0]] - }, "meters" + } ]
\ No newline at end of file |