diff options
| author | Martin Braun <martin.braun@ettus.com> | 2021-01-08 13:23:17 +0100 |
|---|---|---|
| committer | Aaron Rossetto <aaron.rossetto@ni.com> | 2021-03-04 08:24:09 -0600 |
| commit | 47623aabb4832d3d217faa1a9938bb7a6e4341c3 (patch) | |
| tree | b67dec95e65ca8aacd4352413506c8869480e450 /host/lib/usrp/dboard/magnesium/magnesium_radio_control_init.cpp | |
| parent | 49e5ef81530684998bc4bc0934590cdaa81190ab (diff) | |
| download | uhd-47623aabb4832d3d217faa1a9938bb7a6e4341c3.tar.gz uhd-47623aabb4832d3d217faa1a9938bb7a6e4341c3.tar.bz2 uhd-47623aabb4832d3d217faa1a9938bb7a6e4341c3.zip | |
lib: Use const-ref in for loops instead of const-copy
This is potentially a performance issue, even though it doesn't have
a big impact in this context. Clang will warn about it, and this fixes
the compiler warning.
Diffstat (limited to 'host/lib/usrp/dboard/magnesium/magnesium_radio_control_init.cpp')
| -rw-r--r-- | host/lib/usrp/dboard/magnesium/magnesium_radio_control_init.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/host/lib/usrp/dboard/magnesium/magnesium_radio_control_init.cpp b/host/lib/usrp/dboard/magnesium/magnesium_radio_control_init.cpp index 82da7ec75..98c370275 100644 --- a/host/lib/usrp/dboard/magnesium/magnesium_radio_control_init.cpp +++ b/host/lib/usrp/dboard/magnesium/magnesium_radio_control_init.cpp @@ -227,7 +227,7 @@ void magnesium_radio_control_impl::_init_frontend_subtree( // TX gains std::vector<std::string> tx_gain_names = get_tx_gain_names(chan_idx); tx_gain_names.push_back("all"); - for (const auto gain_name : tx_gain_names) { + for (const auto& gain_name : tx_gain_names) { subtree->create<double>(tx_fe_path / "gains" / gain_name / "value") .set_coercer([this, chan_idx, gain_name](const double gain) { return this->set_tx_gain(gain, gain_name, chan_idx); @@ -256,7 +256,7 @@ void magnesium_radio_control_impl::_init_frontend_subtree( // RX gains std::vector<std::string> rx_gain_names = get_rx_gain_names(chan_idx); rx_gain_names.push_back("all"); - for (const auto gain_name : rx_gain_names) { + for (const auto& gain_name : rx_gain_names) { subtree->create<double>(rx_fe_path / "gains" / gain_name / "value") .set_coercer([this, chan_idx, gain_name](const double gain) { return this->set_rx_gain(gain, gain_name, chan_idx); |