Use array of structs rather than parallel arrays for annotations

include/mbgl/annotation/point_annotation.hpp

@@ -0,0 +1,22 @@
+#ifndef MBGL_ANNOTATION_POINT_ANNOTATION
+#define MBGL_ANNOTATION_POINT_ANNOTATION
+
+#include <mbgl/util/geo.hpp>
+
+#include <string>
+
+namespace mbgl {
+
+class PointAnnotation {
+public:
+    inline PointAnnotation(const LatLng& position_, const std::string& icon_ = "")
+        : position(position_), icon(icon_) {
+    }
+
+    const LatLng position;
+    const std::string icon;
+};
+
+}
+
+#endif

include/mbgl/map/map.hpp

@@ -21,6 +21,7 @@ class View;
 class MapData;
 class MapContext;
 class StillImage;
+class PointAnnotation;
 
 namespace util {
 template <class T> class Thread;
@@ -111,9 +112,8 @@ class Map : private util::noncopyable {
     // Annotations
     void setDefaultPointAnnotationSymbol(const std::string&);
     double getTopOffsetPixelsForAnnotationSymbol(const std::string&);
-    uint32_t addPointAnnotation(const LatLng&, const std::string& symbol);
-    std::vector<uint32_t> addPointAnnotations(const std::vector<LatLng>&,
-                                              const std::vector<std::string>& symbols);
+    uint32_t addPointAnnotation(const PointAnnotation&);
+    std::vector<uint32_t> addPointAnnotations(const std::vector<PointAnnotation>&);
     void removeAnnotation(uint32_t);
     void removeAnnotations(const std::vector<uint32_t>&);
     std::vector<uint32_t> getAnnotationsInBounds(const LatLngBounds&);

platform/default/glfw_view.cpp

@@ -1,3 +1,4 @@
+#include <mbgl/annotation/point_annotation.hpp>
 #include <mbgl/platform/default/glfw_view.hpp>
 #include <mbgl/platform/gl.hpp>
 #include <mbgl/platform/log.hpp>
@@ -130,8 +131,7 @@ void GLFWView::onKey(GLFWwindow *window, int key, int /*scancode*/, int action,
 }
 
 void GLFWView::addRandomPointAnnotations(int count) {
-    std::vector<mbgl::LatLng> points;
-    std::vector<std::string> markers;
+    std::vector<mbgl::PointAnnotation> points;
 
     const auto sw = map->latLngForPixel({ 0, 0 });
     const auto ne = map->latLngForPixel({ double(width), double(height) });
@@ -140,11 +140,10 @@ void GLFWView::addRandomPointAnnotations(int count) {
         const double lon = sw.longitude + (ne.longitude - sw.longitude) * (double(std::rand()) / RAND_MAX);
         const double lat = sw.latitude + (ne.latitude - sw.latitude) * (double(std::rand()) / RAND_MAX);
 
-        points.push_back({ lat, lon });
-        markers.push_back("default_marker");
+        points.emplace_back(mbgl::LatLng{ lat, lon }, "default_marker");
     }
 
-    map->addPointAnnotations(points, markers);
+    map->addPointAnnotations(points);
 }
 
 void GLFWView::onScroll(GLFWwindow *window, double /*xOffset*/, double yOffset) {

platform/ios/MGLMapView.mm

@@ -8,6 +8,7 @@
 #import <OpenGLES/EAGL.h>
 
 #include <mbgl/mbgl.hpp>
+#include <mbgl/annotation/point_annotation.hpp>
 #include <mbgl/platform/platform.hpp>
 #include <mbgl/platform/darwin/reachability.h>
 #include <mbgl/storage/default_file_source.hpp>
@@ -1669,31 +1670,26 @@ - (void)addAnnotations:(NSArray *)annotations
 {
     if ( ! annotations) return;
 
-    std::vector<mbgl::LatLng> latLngs;
-    latLngs.reserve(annotations.count);
-
-    std::vector<std::string> symbols;
-    symbols.reserve(annotations.count);
+    std::vector<mbgl::PointAnnotation> points;
+    points.reserve(annotations.count);
 
     BOOL delegateImplementsSymbolLookup = [self.delegate respondsToSelector:@selector(mapView:symbolNameForAnnotation:)];
 
     for (id <MGLAnnotation> annotation in annotations)
     {
         assert([annotation conformsToProtocol:@protocol(MGLAnnotation)]);
 
-        latLngs.push_back(MGLLatLngFromLocationCoordinate2D(annotation.coordinate));
-
         NSString *symbolName = nil;
 
         if (delegateImplementsSymbolLookup)
         {
             symbolName = [self.delegate mapView:self symbolNameForAnnotation:annotation];
         }
 
-        symbols.push_back((symbolName ? [symbolName UTF8String] : ""));
+        points.emplace_back(MGLLatLngFromLocationCoordinate2D(annotation.coordinate), (symbolName ? [symbolName UTF8String] : ""));
     }
 
-    std::vector<uint32_t> annotationIDs = _mbglMap->addPointAnnotations(latLngs, symbols);
+    std::vector<uint32_t> annotationIDs = _mbglMap->addPointAnnotations(points);
 
     for (size_t i = 0; i < annotationIDs.size(); ++i)
     {

src/mbgl/map/annotation.cpp

@@ -85,8 +85,7 @@ vec2<double> AnnotationManager::projectPoint(const LatLng& point) {
 }
 
 std::pair<std::vector<TileID>, AnnotationIDs>
-AnnotationManager::addPointAnnotations(const std::vector<LatLng>& points,
-                                       const std::vector<std::string>& symbols,
+AnnotationManager::addPointAnnotations(const std::vector<PointAnnotation>& points,
                                        const MapData& data) {
     std::lock_guard<std::mutex> lock(mtx);
 
@@ -103,22 +102,22 @@ AnnotationManager::addPointAnnotations(const std::vector<LatLng>& points,
 
     std::vector<TileID> affectedTiles;
 
-    for (size_t i = 0; i < points.size(); ++i) {
+    for (const PointAnnotation& point : points) {
         const uint32_t annotationID = nextID();
 
         // track the annotation global ID and its geometry
         auto anno_it = annotations.emplace(
             annotationID,
             std::make_unique<Annotation>(AnnotationType::Point,
-                                          AnnotationSegments({ { points[i] } })));
+                                          AnnotationSegments({ { point.position } })));
 
         const uint8_t maxZoom = data.transform.getMaxZoom();
 
         // side length of map at this zoom
         uint32_t z2 = 1 << maxZoom;
 
         // projection conversion into unit space
-        const vec2<double> p = projectPoint(points[i]);
+        const vec2<double> p = projectPoint(point.position);
 
         uint32_t x = p.x * z2;
         uint32_t y = p.y * z2;
@@ -134,7 +133,7 @@ AnnotationManager::addPointAnnotations(const std::vector<LatLng>& points,
 
             // at render time we style the annotation according to its {sprite} field
             const std::map<std::string, std::string> properties = {
-                { "sprite", (symbols[i].length() ? symbols[i] : defaultPointAnnotationSymbol) }
+                { "sprite", (point.icon.length() ? point.icon : defaultPointAnnotationSymbol) }
             };
 
             auto feature =

src/mbgl/map/annotation.hpp

@@ -2,6 +2,7 @@
 #define MBGL_MAP_ANNOTATIONS
 
 #include <mbgl/map/tile_id.hpp>
+#include <mbgl/annotation/point_annotation.hpp>
 #include <mbgl/util/geo.hpp>
 #include <mbgl/util/noncopyable.hpp>
 #include <mbgl/util/vec.hpp>
@@ -29,7 +30,7 @@ class AnnotationManager : private util::noncopyable {
 
     void setDefaultPointAnnotationSymbol(const std::string& symbol);
     std::pair<std::vector<TileID>, AnnotationIDs> addPointAnnotations(
-        const std::vector<LatLng>&, const std::vector<std::string>& symbols, const MapData&);
+        const std::vector<PointAnnotation>&, const MapData&);
     std::vector<TileID> removeAnnotations(const AnnotationIDs&, const MapData&);
     AnnotationIDs getAnnotationsInBounds(const LatLngBounds&, const MapData&) const;
     LatLngBounds getBoundsForAnnotations(const AnnotationIDs&) const;

src/mbgl/map/map.cpp

@@ -235,12 +235,12 @@ double Map::getTopOffsetPixelsForAnnotationSymbol(const std::string& symbol) {
     return context->invokeSync<double>(&MapContext::getTopOffsetPixelsForAnnotationSymbol, symbol);
 }
 
-uint32_t Map::addPointAnnotation(const LatLng& point, const std::string& symbol) {
-    return addPointAnnotations({ point }, { symbol }).front();
+uint32_t Map::addPointAnnotation(const PointAnnotation& annotation) {
+    return addPointAnnotations({ annotation }).front();
 }
 
-std::vector<uint32_t> Map::addPointAnnotations(const std::vector<LatLng>& points, const std::vector<std::string>& symbols) {
-    auto result = data->annotationManager.addPointAnnotations(points, symbols, *data);
+std::vector<uint32_t> Map::addPointAnnotations(const std::vector<PointAnnotation>& annotations) {
+    auto result = data->annotationManager.addPointAnnotations(annotations, *data);
     context->invoke(&MapContext::updateAnnotationTiles, result.first);
     return result.second;
 }