From 7e013856740b5f93021bc940ff64411e8607e8fd Mon Sep 17 00:00:00 2001
From: Simon Praetorius <simon.praetorius@tu-dresden.de>
Date: Tue, 8 Feb 2022 14:42:21 +0000
Subject: [PATCH] Use [[maybe_unused]] instead of commenting out the parameters

---
 dune/istl/schwarz.hh | 4 ++--
 dune/istl/solvers.hh | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/dune/istl/schwarz.hh b/dune/istl/schwarz.hh
index 85a81eb3a..894d201ce 100644
--- a/dune/istl/schwarz.hh
+++ b/dune/istl/schwarz.hh
@@ -204,7 +204,7 @@ namespace Dune {
 
        \copydoc Preconditioner::pre(X&,Y&)
      */
-    virtual void pre (X& x, Y& /*b*/)
+    virtual void pre (X& x, [[maybe_unused]] Y& b)
     {
       communication.copyOwnerToAll(x,x);     // make dirichlet values consistent
     }
@@ -228,7 +228,7 @@ namespace Dune {
 
        \copydoc Preconditioner::post(X&)
      */
-    virtual void post (X& /*x*/) {}
+    virtual void post ([[maybe_unused]] X& x) {}
 
     //! Category of the preconditioner (see SolverCategory::Category)
     virtual SolverCategory::Category category() const
diff --git a/dune/istl/solvers.hh b/dune/istl/solvers.hh
index b08dab6db..011e3eaa4 100644
--- a/dune/istl/solvers.hh
+++ b/dune/istl/solvers.hh
@@ -1164,7 +1164,7 @@ namespace Dune {
        \note Currently, the RestartedFlexibleGMResSolver aborts when it detects a
              breakdown.
      */
-    void apply (X& x, Y& b, double /*reduction*/, InverseOperatorResult& res) override
+    void apply (X& x, Y& b, [[maybe_unused]] double reduction, InverseOperatorResult& res) override
     {
       using std::abs;
       const Simd::Scalar<real_type> EPSILON = 1e-80;
@@ -1710,7 +1710,7 @@ private:
     };
 
     // This function is called every mmax iterations to handle limited array sizes.
-    virtual void cycle(std::vector<X>& Ad,std::vector<X>& /*d*/,std::vector<field_type,ReboundAllocatorType<X,field_type> >& /*ddotAd*/,int& i_bounded) override {
+    virtual void cycle(std::vector<X>& Ad, [[maybe_unused]] std::vector<X>& d, [[maybe_unused]] std::vector<field_type,ReboundAllocatorType<X,field_type> >& ddotAd,int& i_bounded) override {
       // Only the loop index i_bounded return to 0, if it reached mmax.
       i_bounded = 0;
       // Now all arrays are filled and the loop in void orthogonalizations can use the whole arrays.
-- 
GitLab