fix warnings, clean NeoFOAM:: in test

exasim-project · Oct 11, 2024 · 9cab253 · 9cab253
1 parent 4791d62
commit 9cab253
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 12 deletions.
diff --git a/include/NeoFOAM/dsl/equation.hpp b/include/NeoFOAM/dsl/equation.hpp
@@ -112,7 +112,7 @@ class Equation
 
     const Executor& exec() const { return exec_; }
 
-    const std::size_t nCells() const { return nCells_; }
+    std::size_t nCells() const { return nCells_; }
 
     scalar getDt() const { return dt_; }
 

diff --git a/include/NeoFOAM/dsl/operator.hpp b/include/NeoFOAM/dsl/operator.hpp
@@ -55,7 +55,7 @@ class OperatorMixin
     const Coeff& getCoefficient() const { return coeffs_; }
 
     /* @brief Given an input this function reads required coeffs */
-    void build(const Input& input) {}
+    void build([[maybe_unused]] const Input& input) {}
 
 protected:
 

diff --git a/include/NeoFOAM/fields/field.hpp b/include/NeoFOAM/fields/field.hpp
@@ -79,7 +79,7 @@ class Field
     {
         void* ptr = nullptr;
         std::visit(
-            [this, &ptr, size](const auto& concreteExec)
+            [&ptr, size](const auto& concreteExec)
             { ptr = concreteExec.alloc(size * sizeof(ValueType)); },
             exec_
         );

diff --git a/test/dsl/coeff.cpp b/test/dsl/coeff.cpp
@@ -62,8 +62,8 @@ TEST_CASE("Coeff")
     {
         size_t size = 3;
 
-        NeoFOAM::Field<NeoFOAM::scalar> fieldA(exec, size, 0.0);
-        NeoFOAM::Field<NeoFOAM::scalar> fieldB(exec, size, 1.0);
+        Field fieldA(exec, size, 0.0);
+        Field fieldB(exec, size, 1.0);
 
         SECTION("span")
         {
@@ -82,8 +82,6 @@ TEST_CASE("Coeff")
 
         SECTION("scalar")
         {
-            NeoFOAM::Field<NeoFOAM::scalar> fieldA(exec, size, 0.0);
-
             Coeff coeff = Coeff(2.0);
             {
                 NeoFOAM::parallelFor(
@@ -99,8 +97,6 @@ TEST_CASE("Coeff")
 
         SECTION("span and scalar")
         {
-            NeoFOAM::Field<NeoFOAM::scalar> fieldA(exec, size, 0.0);
-            NeoFOAM::Field<NeoFOAM::scalar> fieldB(exec, size, 1.0);
             Coeff coeff {-5.0, fieldB};
             {
                 NeoFOAM::parallelFor(

diff --git a/test/dsl/operator.cpp b/test/dsl/operator.cpp
@@ -42,9 +42,9 @@ TEST_CASE("Operator")
         auto d = fB * Dummy(exec, vf);
         auto e = Coeff(-3, fB) * Dummy(exec, vf);
 
-        auto coeffc = c.getCoefficient();
-        auto coeffd = d.getCoefficient();
-        auto coeffE = e.getCoefficient();
+        [[maybe_unused]] auto coeffC = c.getCoefficient();
+        [[maybe_unused]] auto coeffD = d.getCoefficient();
+        [[maybe_unused]] auto coeffE = e.getCoefficient();
 
         Field source(exec, 1, 2.0);
         c.explicitOperation(source);