Browse Source

LinkManager: use range based loop instead when getting links

QGC4.4
davidsastresas 2 years ago committed by David Sastre
parent
commit
e748380e10
  1. 10
      src/comm/LinkManager.cc

10
src/comm/LinkManager.cc

@ -175,10 +175,9 @@ bool LinkManager::createConnectedLink(SharedLinkConfigurationPtr& config, bool i
SharedLinkInterfacePtr LinkManager::mavlinkForwardingLink() SharedLinkInterfacePtr LinkManager::mavlinkForwardingLink()
{ {
for (int i = 0; i < _rgLinks.count(); i++) { for (auto& link : _rgLinks) {
SharedLinkConfigurationPtr linkConfig = _rgLinks[i]->linkConfiguration(); SharedLinkConfigurationPtr linkConfig = link->linkConfiguration();
if (linkConfig->type() == LinkConfiguration::TypeUdp && linkConfig->name() == _mavlinkForwardingLinkName) { if (linkConfig->type() == LinkConfiguration::TypeUdp && linkConfig->name() == _mavlinkForwardingLinkName) {
SharedLinkInterfacePtr& link = _rgLinks[i];
return link; return link;
} }
} }
@ -188,10 +187,9 @@ SharedLinkInterfacePtr LinkManager::mavlinkForwardingLink()
SharedLinkInterfacePtr LinkManager::mavlinkForwardingSupportLink() SharedLinkInterfacePtr LinkManager::mavlinkForwardingSupportLink()
{ {
for (int i = 0; i < _rgLinks.count(); i++) { for (auto& link : _rgLinks) {
SharedLinkConfigurationPtr linkConfig = _rgLinks[i]->linkConfiguration(); SharedLinkConfigurationPtr linkConfig = link->linkConfiguration();
if (linkConfig->type() == LinkConfiguration::TypeUdp && linkConfig->name() == _mavlinkForwardingSupportLinkName) { if (linkConfig->type() == LinkConfiguration::TypeUdp && linkConfig->name() == _mavlinkForwardingSupportLinkName) {
SharedLinkInterfacePtr& link = _rgLinks[i];
return link; return link;
} }
} }

Loading…
Cancel
Save