diff options
| author | John Firebaugh <john.firebaugh@gmail.com> | 2016-11-30 10:17:47 -0800 |
|---|---|---|
| committer | John Firebaugh <john.firebaugh@gmail.com> | 2016-12-01 09:00:06 -0800 |
| commit | 7d16824fda7086fbeb0a1deae3fc0abdf0c4832d (patch) | |
| tree | 11c35642db033c685e78c8b45d28af68a5f3d30b /include/mbgl/style/conversion | |
| parent | 54c557d982c92caefd53a68e2f72bba822101c2c (diff) | |
| download | qtlocation-mapboxgl-7d16824fda7086fbeb0a1deae3fc0abdf0c4832d.tar.gz | |
[core] Functions must have at least one stop
This is already enforced by the style validator in mapbox-gl-style-spec. Enforce it here too.
Diffstat (limited to 'include/mbgl/style/conversion')
| -rw-r--r-- | include/mbgl/style/conversion/function.hpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/include/mbgl/style/conversion/function.hpp b/include/mbgl/style/conversion/function.hpp index f14b5089be..6a0b67618f 100644 --- a/include/mbgl/style/conversion/function.hpp +++ b/include/mbgl/style/conversion/function.hpp @@ -25,6 +25,10 @@ struct Converter<Function<T>> { return Error { "function stops must be an array" }; } + if (arrayLength(*stopsValue) == 0) { + return Error { "function must have at least one stop" }; + } + std::vector<std::pair<float, T>> stops; for (std::size_t i = 0; i < arrayLength(*stopsValue); ++i) { const auto& stopValue = arrayMember(*stopsValue, i); |