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 /src/mbgl | |
parent | af14e1d510649fcd181e4da361803a5caaa5ba62 (diff) | |
download | qtlocation-mapboxgl-4a54da881ff75cc6f034176cb7f249751739e66f.tar.gz |
[core] Distance expression: Remove code and comments related to unit argument
Diffstat (limited to 'src/mbgl')
-rw-r--r-- | src/mbgl/style/expression/distance.cpp | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/src/mbgl/style/expression/distance.cpp b/src/mbgl/style/expression/distance.cpp index 9992ef5394..5ab1206340 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; } |