From 3ec32e441586e97bcc9e20861531d908959a5a11 Mon Sep 17 00:00:00 2001
From: mvglasow <michael -at- vonglasow.com>
Date: Sat, 25 Oct 2025 15:12:05 +0300
Subject: [PATCH] [traffic] Use references for loop, not copies

Signed-off-by: mvglasow <michael -at- vonglasow.com>
---
 libs/traffxml/traff_decoder.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libs/traffxml/traff_decoder.cpp b/libs/traffxml/traff_decoder.cpp
index add330f9e..c8ab1af3f 100644
--- a/libs/traffxml/traff_decoder.cpp
+++ b/libs/traffxml/traff_decoder.cpp
@@ -1075,7 +1075,7 @@ void RoutingTraffDecoder::DecodeLocationDirection(traffxml::TraffMessage & messa
       routing::RouteSegment & closestRSegment = rsegments.front();
       double closestDist = ms::DistanceOnEarth(at, mercator::ToLatLon(closestRSegment.GetJunction().GetPoint()));
 
-      for (auto rsegment : rsegments)
+      for (auto & rsegment : rsegments)
       {
         // If we have more than two checkpoints, fake segments can occur in the middle, skip them.
         if (rsegment.GetSegment().GetMwmId() == routing::kFakeNumMwmId)
@@ -1092,7 +1092,7 @@ void RoutingTraffDecoder::DecodeLocationDirection(traffxml::TraffMessage & messa
     }
     else
       // from–[via]–to, add all real segments
-      for (auto rsegment : rsegments)
+      for (auto & rsegment : rsegments)
       {
         routing::Segment & segment = rsegment.GetSegment();