From aab575263f8338ae027993f8ce8d4fb47e5a3fd0 Mon Sep 17 00:00:00 2001 From: "Thiago Marcos P. Santos" Date: Fri, 27 Mar 2020 18:30:35 +0200 Subject: [PATCH] [core] Fix performance-unnecessary-copy-initialization errors in header files As reported by clang-tidy-8. --- src/mbgl/layout/pattern_layout.hpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mbgl/layout/pattern_layout.hpp b/src/mbgl/layout/pattern_layout.hpp index 8d612d258..b2bcd1b31 100644 --- a/src/mbgl/layout/pattern_layout.hpp +++ b/src/mbgl/layout/pattern_layout.hpp @@ -93,7 +93,7 @@ public: for (const auto& layerProperties : group) { const std::string& layerId = layerProperties->baseImpl->id; const auto& evaluated = style::getEvaluated(layerProperties); - const auto patternProperty = evaluated.template get(); + const auto& patternProperty = evaluated.template get(); const auto constantPattern = patternProperty.constantOr(Faded{"", ""}); // determine if layer group has any layers that use *-pattern property and add // constant pattern dependencies. @@ -122,7 +122,7 @@ public: const auto it = layerPropertiesMap.find(layerId); if (it != layerPropertiesMap.end()) { const auto paint = static_cast(*it->second).evaluated; - const auto patternProperty = paint.template get(); + const auto& patternProperty = paint.template get(); if (!patternProperty.isConstant()) { // For layers with non-data-constant pattern properties, evaluate their expression and add // the patterns to the dependency vector