clean up return after exception

Author: pantor

include/ruckig/calculator_cloud.hpp

@@ -178,9 +178,8 @@ class WaypointsCalculator {
     Result continue_calculation(const InputParameter<DOFs, CustomVector>&, Trajectory<DOFs, CustomVector>&, double, bool&) {
         if constexpr (throw_error) {
             throw RuckigError("continue calculation not available in Ruckig Community Version.");
-        } else {
-            return Result::Error;
         }
+        return Result::Error;
     }
 };
 

include/ruckig/calculator_target.hpp

@@ -311,16 +311,14 @@ class TargetCalculator {
                 if (has_zero_limits) {
                     if constexpr (throw_error) {
                         throw RuckigError("zero limits conflict in step 1, dof: " + std::to_string(dof) + " input: " + inp.to_string());
-                    } else {
-                        return Result::ErrorZeroLimits;
                     }
+                    return Result::ErrorZeroLimits;
 
                 } else {
                     if constexpr (throw_error) {
                         throw RuckigError("error in step 1, dof: " + std::to_string(dof) + " input: " + inp.to_string());
-                    } else {
-                        return Result::ErrorExecutionTimeCalculation;
                     }
+                    return Result::ErrorExecutionTimeCalculation;
                 }
             }
 
@@ -350,16 +348,14 @@ class TargetCalculator {
             if (has_zero_limits) {
                 if constexpr (throw_error) {
                     throw RuckigError("zero limits conflict with other degrees of freedom in time synchronization " + std::to_string(traj.duration));
-                } else {
-                    return Result::ErrorZeroLimits;
                 }
+                return Result::ErrorZeroLimits;
 
             } else {
                 if constexpr (throw_error) {
                     throw RuckigError("error in time synchronization: " + std::to_string(traj.duration));
-                } else {
-                    return Result::ErrorSynchronizationCalculation;
                 }
+                return Result::ErrorSynchronizationCalculation;
             }
         }
 
@@ -515,9 +511,8 @@ class TargetCalculator {
             if (!found_time_synchronization) {
                 if constexpr (throw_error) {
                     throw RuckigError("error in step 2 in dof: " + std::to_string(dof) + " for t sync: " + std::to_string(traj.duration) + " input: " + inp.to_string());
-                } else {
-                    return Result::ErrorSynchronizationCalculation;
                 }
+                return Result::ErrorSynchronizationCalculation;
             }
             // std::cout << dof << " profile step2: " << p.to_string() << std::endl;
         }