diff --git a/src/mbgl/style/layers/symbol_layer_impl.hpp b/src/mbgl/style/layers/symbol_layer_impl.hpp index 9b63e0e8d..ca9505a5f 100644 --- a/src/mbgl/style/layers/symbol_layer_impl.hpp +++ b/src/mbgl/style/layers/symbol_layer_impl.hpp @@ -74,7 +74,7 @@ struct FormatSectionOverrides> { }, [&checkLiteral] (const PropertyExpression& property) { bool expressionHasOverrides = false; - const auto checkExpression = [&](const expression::Expression& e) { + const std::function checkExpression = [&](const expression::Expression& e) { if (expressionHasOverrides) { return; } @@ -87,9 +87,7 @@ struct FormatSectionOverrides> { expressionHasOverrides = true; } return; - } - - if (e.getKind() == expression::Kind::FormatExpression) { + } else if (e.getKind() == expression::Kind::FormatExpression) { const auto* formatExpr = static_cast(&e); for (const auto& section : formatExpr->getSections()) { if (Property::hasOverride(section)) { @@ -97,17 +95,12 @@ struct FormatSectionOverrides> { break; } } + } else { + e.eachChild(checkExpression); } }; - // Check root property expression and return early. checkExpression(property.getExpression()); - if (expressionHasOverrides) { - return true; - } - - // Traverse thru children and check whether any of them have overrides. - property.getExpression().eachChild(checkExpression); return expressionHasOverrides; }, [] (const auto&) {