drifty/comaps
1
0
Fork 0
mirror of https://codeberg.org/comaps/comaps synced 2025-12-19 13:03:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

[traffic] Use references for loop, not copies

Browse Source 
Signed-off-by: mvglasow <michael -at- vonglasow.com>
This commit is contained in:
mvglasow
2025-10-25 15:12:05 +03:00
parent 9065f45b21
commit 3ec32e4415
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
libs/traffxml/traff_decoder.cpp
View File

@@ -1075,7 +1075,7 @@ void RoutingTraffDecoder::DecodeLocationDirection(traffxml::TraffMessage & messa
routing::RouteSegment & closestRSegment = rsegments.front(); routing::RouteSegment & closestRSegment = rsegments.front();
double closestDist = ms::DistanceOnEarth(at, mercator::ToLatLon(closestRSegment.GetJunction().GetPoint())); 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 we have more than two checkpoints, fake segments can occur in the middle, skip them.
if (rsegment.GetSegment().GetMwmId() == routing::kFakeNumMwmId) if (rsegment.GetSegment().GetMwmId() == routing::kFakeNumMwmId)
@@ -1092,7 +1092,7 @@ void RoutingTraffDecoder::DecodeLocationDirection(traffxml::TraffMessage & messa
} }
else else
// from–[via]–to, add all real segments // from–[via]–to, add all real segments
for (auto rsegment : rsegments) for (auto & rsegment : rsegments)
{ {
routing::Segment & segment = rsegment.GetSegment(); routing::Segment & segment = rsegment.GetSegment();