From 3ce3fae91e702f1d7ee3265bf4dd314fc1060699 Mon Sep 17 00:00:00 2001 From: Daniel <d.hornung@indiscale.com> Date: Thu, 19 Aug 2021 08:17:16 +0200 Subject: [PATCH] STY: adding a few consts --- src/caosdb/utility.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/caosdb/utility.cpp b/src/caosdb/utility.cpp index 243e85e..6206396 100644 --- a/src/caosdb/utility.cpp +++ b/src/caosdb/utility.cpp @@ -63,7 +63,7 @@ auto getEnumValueFromName<Importance>(const std::string &name) -> Importance { // caosdb::entity::importance_names.cend(), // [name](const auto& entry){ return entry.second == name; }); // Workaround: plaint old iteration: - for (auto entry: caosdb::entity::importance_names) { + for (auto const &entry: caosdb::entity::importance_names) { if (entry.second == name) { return entry.first; } @@ -73,16 +73,17 @@ auto getEnumValueFromName<Importance>(const std::string &name) -> Importance { template <> auto getEnumValueFromName<AtomicDataType>(const std::string &name) -> AtomicDataType { - for (auto entry: caosdb::entity::atomicdatatype_names) { + for (auto const &entry: caosdb::entity::atomicdatatype_names) { if (entry.second == name) { return entry.first; } } throw std::out_of_range(std::string("Could not find enum value for string '") + name + "'."); } + template <> auto getEnumValueFromName<Role>(const std::string &name) -> Role { - for (auto entry: caosdb::entity::role_names) { + for (auto const &entry: caosdb::entity::role_names) { if (entry.second == name) { return entry.first; } -- GitLab