Apply suggestions from giudgiud code review 1 Refs #28891

Co-authored-by: Guillaume Giudicelli <guillaume.giudicelli@gmail.com>
idaholab · Oct 29, 2024 · 1bc9eb5 · 1bc9eb5
1 parent 15559c5
commit 1bc9eb5
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/framework/include/limiters/Limiter.h b/framework/include/limiters/Limiter.h
@@ -128,7 +128,7 @@ class Limiter
    * @brief Functor for applying simplified slope limiting.
    *
    * This function applies the limiter by invoking the `limit` method with the provided parameters.
-   * It acts as a functor, enabling objects of the `VanLeerLimiter` class to be used as if they were
+   * It acts as a functor, enabling objects of the derived `Limiter` class to be used as if they were
    * functions.
    *
    * @tparam T The data type of the scalar values and the return type.

diff --git a/framework/include/variables/MooseVariableFV.h b/framework/include/variables/MooseVariableFV.h
@@ -653,7 +653,7 @@ class MooseVariableFV : public MooseVariableField<OutputType>
                                                   const Elem * elem_side_to_extrapolate_from,
                                                   const StateArg & state) const;
 
-  /// Function to get the two term boudnary exansion for the variable
+  /// Function to get the two term boundary expansion for the variable
   const bool & getTwoTermBoundaryExpansion() const { return _two_term_boundary_expansion; }
 
 protected:

diff --git a/modules/navier_stokes/src/fvkernels/INSFVAdvectionKernel.C b/modules/navier_stokes/src/fvkernels/INSFVAdvectionKernel.C
@@ -62,8 +62,8 @@ INSFVAdvectionKernel::INSFVAdvectionKernel(const InputParameters & params)
         _advected_interp_method == Moose::FV::InterpMethod::Average))
     mooseWarning("Second order upwind limiting is not supported when `two_term_boundary_expansion "
                  "= true` for the limited variable. Use at your own risk or please consider "
-                 "changing `two_term_boundary_expansion = false` or changing your "
-                 "advected_interp_method to first order methods (`upwind`, `average`)");
+                 "setting `two_term_boundary_expansion = false` in the advected variable parameters or changing your "
+                 "'advected_interp_method' of the kernel to first order methods (`upwind`, `average`)");
 }
 
 void