From 4951433330f66b6faa5ab3cc512171cf0cda5653 Mon Sep 17 00:00:00 2001 From: Alexey Rybalchenko Date: Thu, 6 Sep 2018 17:53:54 +0200 Subject: [PATCH] Apply clang-tidy suggestions [modernize-loop-convert] --- fairmq/FairMQDevice.cxx | 24 ++++++++++++------------ fairmq/shmem/Monitor.cxx | 4 ++-- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/fairmq/FairMQDevice.cxx b/fairmq/FairMQDevice.cxx index d8a3e804..c654c5d1 100644 --- a/fairmq/FairMQDevice.cxx +++ b/fairmq/FairMQDevice.cxx @@ -471,15 +471,15 @@ void FairMQDevice::PrintChannel(const string& name) { if (fChannels.find(name) != fChannels.end()) { - for (auto vi = fChannels[name].begin(); vi != fChannels[name].end(); ++vi) + for (const auto& vi : fChannels[name]) { - LOG(info) << vi->fName << ": " - << vi->fType << " | " - << vi->fMethod << " | " - << vi->fAddress << " | " - << vi->fSndBufSize << " | " - << vi->fRcvBufSize << " | " - << vi->fRateLogging; + LOG(info) << vi.fName << ": " + << vi.fType << " | " + << vi.fMethod << " | " + << vi.fAddress << " | " + << vi.fSndBufSize << " | " + << vi.fRcvBufSize << " | " + << vi.fRateLogging; } } else @@ -596,17 +596,17 @@ void FairMQDevice::HandleMultipleChannelInput() for (const auto& mi : fMsgInputs) { - for (unsigned int i = 0; i < fChannels.at(mi.first).size(); ++i) + for (auto& i : fChannels.at(mi.first)) { - fChannels.at(mi.first).at(i).fMultipart = false; + i.fMultipart = false; } } for (const auto& mi : fMultipartInputs) { - for (unsigned int i = 0; i < fChannels.at(mi.first).size(); ++i) + for (auto& i : fChannels.at(mi.first)) { - fChannels.at(mi.first).at(i).fMultipart = true; + i.fMultipart = true; } } diff --git a/fairmq/shmem/Monitor.cxx b/fairmq/shmem/Monitor.cxx index 97bd7dbf..eb7dae1f 100644 --- a/fairmq/shmem/Monitor.cxx +++ b/fairmq/shmem/Monitor.cxx @@ -443,9 +443,9 @@ void Monitor::PrintQueues() { cout << "found " << queues->size() << " queue(s):" << endl; - for (unsigned int i = 0; i < queues->size(); ++i) + for (const auto& queue : *queues) { - string name(queues->at(i).c_str()); + string name(queue.c_str()); cout << '\t' << name << " : "; atomic* queueSize = segment.find>(name.c_str()).first; if (queueSize)