From e748380e1045109a66850ca488dd4cfb09f1e67d Mon Sep 17 00:00:00 2001 From: davidsastresas Date: Sat, 3 Jun 2023 13:27:10 +0200 Subject: [PATCH] LinkManager: use range based loop instead when getting links --- src/comm/LinkManager.cc | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/comm/LinkManager.cc b/src/comm/LinkManager.cc index 529584d..1215516 100644 --- a/src/comm/LinkManager.cc +++ b/src/comm/LinkManager.cc @@ -175,10 +175,9 @@ bool LinkManager::createConnectedLink(SharedLinkConfigurationPtr& config, bool i SharedLinkInterfacePtr LinkManager::mavlinkForwardingLink() { - for (int i = 0; i < _rgLinks.count(); i++) { - SharedLinkConfigurationPtr linkConfig = _rgLinks[i]->linkConfiguration(); + for (auto& link : _rgLinks) { + SharedLinkConfigurationPtr linkConfig = link->linkConfiguration(); if (linkConfig->type() == LinkConfiguration::TypeUdp && linkConfig->name() == _mavlinkForwardingLinkName) { - SharedLinkInterfacePtr& link = _rgLinks[i]; return link; } } @@ -188,10 +187,9 @@ SharedLinkInterfacePtr LinkManager::mavlinkForwardingLink() SharedLinkInterfacePtr LinkManager::mavlinkForwardingSupportLink() { - for (int i = 0; i < _rgLinks.count(); i++) { - SharedLinkConfigurationPtr linkConfig = _rgLinks[i]->linkConfiguration(); + for (auto& link : _rgLinks) { + SharedLinkConfigurationPtr linkConfig = link->linkConfiguration(); if (linkConfig->type() == LinkConfiguration::TypeUdp && linkConfig->name() == _mavlinkForwardingSupportLinkName) { - SharedLinkInterfacePtr& link = _rgLinks[i]; return link; } }