Merge pull request #138 from xylar/omega/critical-error-on-bad-time-stepper

Add a critical error if TimeStepper is invalid

Author: xylar

components/omega/src/timeStepping/TimeStepper.cpp

@@ -17,13 +17,19 @@ std::map<std::string, std::unique_ptr<TimeStepper>>
 // convert string into TimeStepperType enum
 TimeStepperType getTimeStepperFromStr(const std::string &InString) {
 
-   TimeStepperType TimeStepperChoice;
+   // Initialize TimeStepperChoice with Invalid
+   TimeStepperType TimeStepperChoice = TimeStepperType::Invalid;
+
    if (InString == "Forward-Backward") {
       TimeStepperChoice = TimeStepperType::ForwardBackward;
    } else if (InString == "RungeKutta4") {
       TimeStepperChoice = TimeStepperType::RungeKutta4;
    } else if (InString == "RungeKutta2") {
       TimeStepperChoice = TimeStepperType::RungeKutta2;
+   } else {
+      LOG_CRITICAL("TimeStepper should be one of 'Forward-Backward', "
+                   "'RungeKutta4' or 'RungeKutta2' but got {}:",
+                   InString);
    }
 
    return TimeStepperChoice;

components/omega/src/timeStepping/TimeStepper.h

@@ -27,7 +27,12 @@ namespace OMEGA {
 
 /// An enum for every time stepper type
 /// needs to extended every time a new time stepper is added
-enum class TimeStepperType { ForwardBackward, RungeKutta4, RungeKutta2 };
+enum class TimeStepperType {
+   ForwardBackward,
+   RungeKutta4,
+   RungeKutta2,
+   Invalid
+};
 
 /// Translate string for time stepper type into enum
 TimeStepperType getTimeStepperFromStr(