Fix clang-tidy

sxs-collaboration · Jul 10, 2024 · 09bfeb3 · 09bfeb3
1 parent dc73e44
commit 09bfeb3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/src/Evolution/Systems/ScalarTensor/ConstraintDamping/ConstraintGammas.hpp b/src/Evolution/Systems/ScalarTensor/ConstraintDamping/ConstraintGammas.hpp
@@ -24,12 +24,10 @@
 namespace Tags {
 struct Time;
 }  // namespace Tags
-namespace domain {
-namespace Tags {
+namespace domain::Tags {
 template <size_t Dim, typename Frame>
 struct Coordinates;
-}  // namespace Tags
-}  // namespace domain
+}  // namespace domain::Tags
 class DataVector;
 template <typename X, typename Symm, typename IndexList>
 class Tensor;
@@ -53,7 +51,7 @@ struct ConstraintGamma1Compute : ::CurvedScalarWave::Tags::ConstraintGamma1,
   using return_type = Scalar<DataVector>;
 
   static constexpr void function(
-      const gsl::not_null<Scalar<DataVector>*> gamma1,
+      gsl::not_null<Scalar<DataVector>*> gamma1,
       const ::ScalarTensor::ConstraintDamping::DampingFunction<
           SpatialDim, Frame>& damping_function,
       const tnsr::I<DataVector, SpatialDim, Frame>& coords, const double time,
@@ -84,7 +82,7 @@ struct ConstraintGamma2Compute : ::CurvedScalarWave::Tags::ConstraintGamma2,
   using return_type = Scalar<DataVector>;
 
   static constexpr void function(
-      const gsl::not_null<Scalar<DataVector>*> gamma,
+      gsl::not_null<Scalar<DataVector>*> gamma,
       const ::ScalarTensor::ConstraintDamping::DampingFunction<
           SpatialDim, Frame>& damping_function,
       const tnsr::I<DataVector, SpatialDim, Frame>& coords, const double time,

diff --git a/src/Evolution/Systems/ScalarTensor/ConstraintDamping/DampingFunction.hpp b/src/Evolution/Systems/ScalarTensor/ConstraintDamping/DampingFunction.hpp
@@ -60,14 +60,14 @@ class DampingFunction : public PUP::able {
   /// @{
   /// Returns the value of the function at the coordinate 'x'.
   virtual void operator()(
-      const gsl::not_null<Scalar<double>*> value_at_x,
+      gsl::not_null<Scalar<double>*> value_at_x,
       const tnsr::I<double, VolumeDim, Fr>& x, double time,
       const std::unordered_map<
           std::string,
           std::unique_ptr<domain::FunctionsOfTime::FunctionOfTime>>&
           functions_of_time) const = 0;
   virtual void operator()(
-      const gsl::not_null<Scalar<DataVector>*> value_at_x,
+      gsl::not_null<Scalar<DataVector>*> value_at_x,
       const tnsr::I<DataVector, VolumeDim, Fr>& x, double time,
       const std::unordered_map<
           std::string,