From 3c29614ed1452868909840fc5307652ecc26d295 Mon Sep 17 00:00:00 2001 From: David Robillard Date: Sun, 9 Feb 2020 14:41:18 +0100 Subject: Use range-based for loops in more places --- src/Configuration.cpp | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) (limited to 'src/Configuration.cpp') diff --git a/src/Configuration.cpp b/src/Configuration.cpp index 0d6cfc0..0c86e3e 100644 --- a/src/Configuration.cpp +++ b/src/Configuration.cpp @@ -290,11 +290,10 @@ Configuration::save() // Try to find a writable configuration file const std::vector filenames = get_filenames(); std::ofstream file; - for (size_t i = 0; i < filenames.size(); ++i) { - file.open(filenames[i].c_str(), std::ios::out); + for (const std::string& filename : filenames) { + file.open(filename.c_str(), std::ios::out); if (file.good()) { - std::cout << "Writing configuration to " << filenames[i] - << std::endl; + std::cout << "Writing configuration to " << filename << std::endl; break; } } @@ -325,12 +324,9 @@ Configuration::save() } file << std::dec << std::nouppercase; - for (std::map::iterator i = - _module_settings.begin(); - i != _module_settings.end(); - ++i) { - const ModuleSettings& settings = (*i).second; - const std::string& name = (*i).first; + for (const auto& s : _module_settings) { + const std::string& name = s.first; + const ModuleSettings& settings = s.second; if (settings.split) { if (settings.input_location && settings.output_location) { -- cgit v1.2.1