Fix more -Wrange-loop-analysis findings

diff --git a/include/deal.II/base/index_set.h b/include/deal.II/base/index_set.h
index 42dd03323e3b3281df989fefefc3b1aa94eddf6a..68a7cad4a9885bdd92218633b1d2778603c5c8ad 100644 (file)
--- a/include/deal.II/base/index_set.h
+++ b/include/deal.II/base/index_set.h
@@ -691,7 +691,7 @@ public:
      * Dereferencing operator. The returned value is the index of the element
      * inside the IndexSet.
      */
-    size_type operator*() const;
+    const size_type &operator*() const;
 
     /**
      * Does this iterator point to an existing element?
@@ -1283,7 +1283,7 @@ IndexSet::ElementIterator::is_valid() const
 
 
 
-inline IndexSet::size_type IndexSet::ElementIterator::operator*() const
+inline const IndexSet::size_type &IndexSet::ElementIterator::operator*() const
 {
   Assert(
     is_valid(),

diff --git a/source/numerics/error_estimator_1d.cc b/source/numerics/error_estimator_1d.cc
index f4aad3e0629a502ed6316e3783ad06520b2951d3..8c670368be49fe20c3bdb0f5e360385b6e50532f 100644 (file)
--- a/source/numerics/error_estimator_1d.cc
+++ b/source/numerics/error_estimator_1d.cc
@@ -348,7 +348,7 @@ KellyErrorEstimator<1, spacedim>::estimate(
                     "indicator for internal boundaries in your boundary "
                     "value map."));
 
-  for (const auto boundary_function : neumann_bc)
+  for (const auto &boundary_function : neumann_bc)
     {
       (void)boundary_function;
       Assert(boundary_function.second->n_components == n_components,