From 142b1f18eb1ea185abc5bcad09570d4e999518d9 Mon Sep 17 00:00:00 2001
From: Jason Wray <jason@mapbox.com>
Date: Fri, 1 Dec 2017 19:23:31 -0500
Subject: [core] Fix copy in range-based for loops

Caught by `CLANG_WARN_RANGE_LOOP_ANALYSIS = YES`.
---
 include/mbgl/style/expression/interpolate.hpp | 2 +-
 include/mbgl/style/function/convert.hpp       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/mbgl/style/expression/interpolate.hpp b/include/mbgl/style/expression/interpolate.hpp
index 2dcb5a32a4..439122f91c 100644
--- a/include/mbgl/style/expression/interpolate.hpp
+++ b/include/mbgl/style/expression/interpolate.hpp
@@ -73,7 +73,7 @@ public:
 
     void eachChild(const std::function<void(const Expression&)>& visit) const override {
         visit(*input);
-        for (const std::pair<const double, const std::unique_ptr<Expression>&>& stop : stops) {
+        for (const auto& stop : stops) {
             visit(*stop.second);
         }
     }
diff --git a/include/mbgl/style/function/convert.hpp b/include/mbgl/style/function/convert.hpp
index ed35b4bf14..9f7b7ed1f8 100644
--- a/include/mbgl/style/function/convert.hpp
+++ b/include/mbgl/style/function/convert.hpp
@@ -174,7 +174,7 @@ struct Convert {
     template <typename T>
     static std::map<double, std::unique_ptr<Expression>> convertStops(const std::map<float, T>& stops) {
         std::map<double, std::unique_ptr<Expression>> convertedStops;
-        for(const std::pair<float, T>& stop : stops) {
+        for(const auto& stop : stops) {
             convertedStops.emplace(
                 stop.first,
                 makeLiteral(stop.second)
-- 
cgit v1.2.1