From 516a74eac110fa95ae275a7666133cbb8a375e1e Mon Sep 17 00:00:00 2001
From: Daniel Arndt <daniel.arndt@iwr.uni-heidelberg.de>
Date: Sat, 25 Aug 2018 19:41:39 +0200
Subject: [PATCH] Don't pass const bool by reference

---
 include/deal.II/base/parameter_handler.h | 2 +-
 include/deal.II/sundials/ida.h           | 2 +-
 include/deal.II/sundials/kinsol.h        | 2 +-
 source/base/parameter_handler.cc         | 2 +-
 source/grid/grid_tools.cc                | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/include/deal.II/base/parameter_handler.h b/include/deal.II/base/parameter_handler.h
index 2a98bf5ca2..2cf6b492f1 100644
--- a/include/deal.II/base/parameter_handler.h
+++ b/include/deal.II/base/parameter_handler.h
@@ -1317,7 +1317,7 @@ public:
    * the new value does not conform to the pattern for this entry.
    */
   void
-  set(const std::string &entry_name, const bool &new_value);
+  set(const std::string &entry_name, const bool new_value);
 
 
   /**
diff --git a/include/deal.II/sundials/ida.h b/include/deal.II/sundials/ida.h
index ad48bd2ec5..66de1f6461 100644
--- a/include/deal.II/sundials/ida.h
+++ b/include/deal.II/sundials/ida.h
@@ -315,7 +315,7 @@ namespace SUNDIALS
         // Error parameters
         const double &absolute_tolerance                = 1e-6,
         const double &relative_tolerance                = 1e-5,
-        const bool &  ignore_algebraic_terms_for_errors = true,
+        const bool    ignore_algebraic_terms_for_errors = true,
         // Initial conditions parameters
         const InitialConditionCorrection &ic_type    = use_y_diff,
         const InitialConditionCorrection &reset_type = use_y_diff,
diff --git a/include/deal.II/sundials/kinsol.h b/include/deal.II/sundials/kinsol.h
index 0c7dfab7c3..16681e232a 100644
--- a/include/deal.II/sundials/kinsol.h
+++ b/include/deal.II/sundials/kinsol.h
@@ -262,7 +262,7 @@ namespace SUNDIALS
         const unsigned int      maximum_non_linear_iterations = 200,
         const double &          function_tolerance            = 0.0,
         const double &          step_tolerance                = 0.0,
-        const bool &            no_init_setup                 = false,
+        const bool              no_init_setup                 = false,
         const unsigned int      maximum_setup_calls           = 0,
         const double &          maximum_newton_step           = 0.0,
         const double &          dq_relative_error             = 0.0,
diff --git a/source/base/parameter_handler.cc b/source/base/parameter_handler.cc
index 780d1bbe76..54d4e4619e 100644
--- a/source/base/parameter_handler.cc
+++ b/source/base/parameter_handler.cc
@@ -1031,7 +1031,7 @@ ParameterHandler::set(const std::string &entry_string, const long int new_value)
 
 
 void
-ParameterHandler::set(const std::string &entry_string, const bool &new_value)
+ParameterHandler::set(const std::string &entry_string, const bool new_value)
 {
   // hand this off to the function that
   // actually sets the value as a string
diff --git a/source/grid/grid_tools.cc b/source/grid/grid_tools.cc
index 10498843ba..48f77b7a03 100644
--- a/source/grid/grid_tools.cc
+++ b/source/grid/grid_tools.cc
@@ -4425,7 +4425,7 @@ namespace GridTools
                      std::vector<Point<spacedim>>,
                      std::vector<unsigned int>>,
           cell_hash<dim, spacedim>> &output_unmap,
-        const bool &                 check_owned)
+        const bool                   check_owned)
       {
         bool no_check = !check_owned;
 
-- 
2.39.5