From: Daniel Arndt <arndtd@ornl.gov>
Date: Fri, 18 Oct 2019 14:53:26 +0000 (-0400)
Subject: Apply suggestions from code review
X-Git-Tag: v9.2.0-rc1~966^2~1
X-Git-Url: https://gitweb.dealii.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e0db6fdac07a0953329910b4dbdfd24188738f52;p=dealii.git

Apply suggestions from code review

Co-Authored-By: Wolfgang Bangerth <bangerth@colostate.edu>
---

diff --git a/examples/step-13/step-13.cc b/examples/step-13/step-13.cc
index c80a6ca6eb..688d598e71 100644
--- a/examples/step-13/step-13.cc
+++ b/examples/step-13/step-13.cc
@@ -797,7 +797,7 @@ namespace Step13
       // capture default or to list all the variables that are to be bound to
       // the lambda explicitly. For the sake of clarity we decided to omit
       // the capture default here, but that capture list could equally well be
-      // `[&]` meaning that all used variables are copied into the lambda by
+      // `[&]`, meaning that all used variables are copied into the lambda by
       // reference.
       //
       // At this point, we have assembled the matrix and condensed
diff --git a/examples/step-14/step-14.cc b/examples/step-14/step-14.cc
index 08a9f07a07..4f73f70132 100644
--- a/examples/step-14/step-14.cc
+++ b/examples/step-14/step-14.cc
@@ -2201,7 +2201,8 @@ namespace Step14
           cell, scratch_data, copy_data, error_indicators, face_integrals);
       };
 
-      auto copier = std::function<void(const WeightedResidualCopyData &)>();
+      auto do_nothing_copier =
+        std::function<void(const WeightedResidualCopyData &)>();
 
       // Then hand it all off to WorkStream::run() to compute the
       // estimators for all cells in parallel:
@@ -2209,7 +2210,7 @@ namespace Step14
         DualSolver<dim>::dof_handler.begin_active(),
         DualSolver<dim>::dof_handler.end(),
         worker,
-        copier,
+        do_nothing_copier,
         WeightedResidualScratchData(*DualSolver<dim>::fe,
                                     *DualSolver<dim>::quadrature,
                                     *DualSolver<dim>::face_quadrature,
diff --git a/examples/step-32/step-32.cc b/examples/step-32/step-32.cc
index 42ad54437b..9f402a3769 100644
--- a/examples/step-32/step-32.cc
+++ b/examples/step-32/step-32.cc
@@ -2728,7 +2728,7 @@ namespace Step32
                                              data);
       };
 
-    auto copier = [=](const Assembly::CopyData::TemperatureRHS<dim> &data) {
+    auto copier = [this](const Assembly::CopyData::TemperatureRHS<dim> &data) {
       this->copy_local_to_global_temperature_rhs(data);
     };