[core] Use fixed-size std::array for ImageSource coordinates

3 files changed, 11 insertions, 12 deletions

diff --git a/include/mbgl/style/conversion/coordinate.hpp b/include/mbgl/style/conversion/coordinate.hpp
index 736f5e94a2..732624e77f 100644
--- a/include/mbgl/style/conversion/coordinate.hpp
+++ b/include/mbgl/style/conversion/coordinate.hpp
@@ -13,7 +13,7 @@ public:
     template <class V>
     optional<LatLng> operator() (const V& value, Error& error) const {
         if (!isArray(value) || arrayLength(value) < 2 ) {
-            error = { "coordinate array must contain numeric longtitude and latitude values" };
+            error = { "coordinate array must contain numeric longitude and latitude values" };
             return {};
         }
         //Style spec uses GeoJSON convention for specifying coordinates
@@ -21,7 +21,7 @@ public:
         optional<double> longitude = toDouble(arrayMember(value, 0));
     
         if (!latitude || !longitude) {
-            error = { "coordinate array must contain numeric longtitude and latitude values" };
+            error = { "coordinate array must contain numeric longitude and latitude values" };
             return {};
         }
         if (*latitude < -90 || *latitude > 90 ){
diff --git a/include/mbgl/style/conversion/source.hpp b/include/mbgl/style/conversion/source.hpp
index e121e36955..e0563ac10b 100644
--- a/include/mbgl/style/conversion/source.hpp
+++ b/include/mbgl/style/conversion/source.hpp
@@ -170,14 +170,13 @@ private:
             return {};
         }
         
-        std::vector<LatLng> coordinates;
-        coordinates.reserve(4);
-        for( std::size_t i=0; i < arrayLength(*coordinatesValue); i++) {
+        std::array<LatLng, 4> coordinates;
+        for (std::size_t i=0; i < 4; i++) {
             auto latLng = conversion::convert<LatLng>(arrayMember(*coordinatesValue,i), error);
             if (!latLng) {
                 return {};
             }
-            coordinates.push_back(*latLng);
+            coordinates[i] = *latLng;
         }
         auto result = std::make_unique<ImageSource>(id, coordinates);
         result->setURL(*urlString);
diff --git a/include/mbgl/style/sources/image_source.hpp b/include/mbgl/style/sources/image_source.hpp
index fd67ad38e5..d8a2c45bd8 100644
--- a/include/mbgl/style/sources/image_source.hpp
+++ b/include/mbgl/style/sources/image_source.hpp
@@ -12,16 +12,16 @@ namespace style {
 
 class ImageSource : public Source {
 public:
-    ImageSource(std::string id, const std::vector<LatLng>);
+    ImageSource(std::string id, const std::array<LatLng, 4>);
     ~ImageSource() override;
 
-    const std::string& getURL() const;
-    void setURL(const std::string& url) ;
+    optional<std::string> getURL() const;
+    void setURL(const std::string& url);
 
-    void setImage(mbgl::UnassociatedImage&&);
+    void setImage(UnassociatedImage&&);
 
-    void setCoordinates(const std::vector<LatLng>&);
-    std::vector<LatLng> getCoordinates() const;
+    void setCoordinates(const std::array<LatLng, 4>&);
+    std::array<LatLng, 4> getCoordinates() const;
 
     class Impl;
     const Impl& impl() const;