Unnecessary 'else'.

[simgrid.git] / src / plugins / host_dvfs.cpp

diff --git a/src/plugins/host_dvfs.cpp b/src/plugins/host_dvfs.cpp
index 78d051f..e4f1d01 100644 (file)
--- a/src/plugins/host_dvfs.cpp
+++ b/src/plugins/host_dvfs.cpp
@@ -391,25 +391,20 @@ static void on_host_added(simgrid::s4u::Host& host)
     }
 
     auto governor = [&dvfs_governor, &daemon_proc]() -> std::unique_ptr<simgrid::plugin::dvfs::Governor> {
-      if (dvfs_governor == "conservative") {
+      if (dvfs_governor == "conservative")
         return std::make_unique<simgrid::plugin::dvfs::Conservative>(daemon_proc->get_host());
-      } else if (dvfs_governor == "ondemand") {
+      if (dvfs_governor == "ondemand")
         return std::make_unique<simgrid::plugin::dvfs::OnDemand>(daemon_proc->get_host());
-      }
 #if HAVE_SMPI
-      else if (dvfs_governor == "adagio") {
+      if (dvfs_governor == "adagio")
         return std::make_unique<simgrid::plugin::dvfs::Adagio>(daemon_proc->get_host());
-      }
 #endif
-      else if (dvfs_governor == "performance") {
-        return std::make_unique<simgrid::plugin::dvfs::Performance>(daemon_proc->get_host());
-      } else if (dvfs_governor == "powersave") {
+      if (dvfs_governor == "powersave")
         return std::make_unique<simgrid::plugin::dvfs::Powersave>(daemon_proc->get_host());
-      } else {
+      if (dvfs_governor != "performance")
         XBT_CRITICAL("No governor specified for host %s, falling back to Performance",
                      daemon_proc->get_host()->get_cname());
-        return std::make_unique<simgrid::plugin::dvfs::Performance>(daemon_proc->get_host());
-      }
+      return std::make_unique<simgrid::plugin::dvfs::Performance>(daemon_proc->get_host());
     }();
 
     while (true) {