From 9a7af86a5e92c630f898d8a01e0c60b435aaea18 Mon Sep 17 00:00:00 2001
From: Roland Rossgotterer <roland.rossgotterer@gmail.com>
Date: Mon, 7 Jan 2019 16:23:33 +0100
Subject: [PATCH] Loop over the container itself instead of a temporary value
 list

---
 src/plugins/simulator/xplane/simulatorxplane.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/plugins/simulator/xplane/simulatorxplane.cpp b/src/plugins/simulator/xplane/simulatorxplane.cpp
index fcfd25397..feedd1158 100644
--- a/src/plugins/simulator/xplane/simulatorxplane.cpp
+++ b/src/plugins/simulator/xplane/simulatorxplane.cpp
@@ -791,15 +791,13 @@ namespace BlackSimPlugin
             const qint64 currentTimestamp = QDateTime::currentMSecsSinceEpoch();
 
             // interpolation for all remote aircraft
-            const QList<CXPlaneMPAircraft> xplaneAircraftList(m_xplaneAircraftObjects.values());
-
             PlanesPositions planesPositions;
             PlanesSurfaces planesSurfaces;
             PlanesTransponders planesTransponders;
 
             int aircraftNumber = 0;
             const bool updateAllAircraft = this->isUpdateAllRemoteAircraft(currentTimestamp);
-            for (const CXPlaneMPAircraft &xplaneAircraft : xplaneAircraftList)
+            for (const CXPlaneMPAircraft &xplaneAircraft : m_xplaneAircraftObjects)
             {
                 const CCallsign callsign(xplaneAircraft.getCallsign());
                 Q_ASSERT_X(!callsign.isEmpty(), Q_FUNC_INFO, "missing callsign");