[TimeInterval] Changed the name and simplified the body of a member function

NumLib/TimeStepping/TimeInterval.h

@@ -18,7 +18,7 @@ namespace NumLib
  * Class for a time interval, which has a member to check whether the given time
  * is in this time interval.
  */
-class TimeInterval
+class TimeInterval final
 {
 public:
     TimeInterval(const double start_time, const double end_time)
@@ -26,11 +26,9 @@ public:
     {
     }
 
-    bool isInThisTimeInterval(const double current_time) const
+    bool contains(const double current_time) const
     {
-        return (current_time >= _start_time && current_time <= _end_time)
-                   ? true
-                   : false;
+        return (current_time >= _start_time && current_time <= _end_time);
     }
 
 private:

ProcessLib/BoundaryCondition/DirichletBoundaryCondition.cpp

@@ -70,7 +70,7 @@ void DirichletBoundaryCondition::getEssentialBCValues(
     const double t, GlobalVector const& x,
     NumLib::IndexValueVector<GlobalIndexType>& bc_values) const
 {
-    return getEssentialBCValuesLocal(t, x, bc_values);
+    getEssentialBCValuesLocal(t, x, bc_values);
 }
 
 void DirichletBoundaryCondition::getEssentialBCValuesLocal(

ProcessLib/BoundaryCondition/DirichletBoundaryConditionWithinTimeInterval.cpp

@@ -40,9 +40,10 @@ void DirichletBoundaryConditionWithinTimeInterval::getEssentialBCValues(
     const double t, GlobalVector const& x,
     NumLib::IndexValueVector<GlobalIndexType>& bc_values) const
 {
-    if (_time_interval->isInThisTimeInterval(t))
+    if (_time_interval->contains(t))
     {
-        return getEssentialBCValuesLocal(t, x, bc_values);
+        getEssentialBCValuesLocal(t, x, bc_values);
+        return;
     }
 
     bc_values.ids.clear();