]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Use better variable names in step-69. 14861/head
authorWolfgang Bangerth <bangerth@colostate.edu>
Thu, 9 Mar 2023 03:11:35 +0000 (20:11 -0700)
committerWolfgang Bangerth <bangerth@colostate.edu>
Thu, 9 Mar 2023 03:11:35 +0000 (20:11 -0700)
examples/step-69/step-69.cc

index c013a95546c01d5ee39a13cd9f145562e9493772..92f9e3951cfa86330795cdfed34ecaa5154e8813 100644 (file)
@@ -1222,9 +1222,10 @@ namespace Step69
     // - The iterator <code>indices.begin()</code> points to a row index.
     // - The iterator <code>indices.end()</code> points to a numerically higher
     //   row index.
-    // - The function <code>on_subranges(i1,i2)</code> (where <code>i1</code>
-    //   and <code>i2</code> define a sub-range within the range spanned by
-    //   the end and begin iterators defined in the two previous bullets)
+    // - The function <code>on_subranges(index_begin, index_end)</code>
+    //   (where <code>index_begin</code> and <code>index_end</code>
+    //   define a sub-range within the range spanned by
+    //   the begin and end iterators defined in the two previous bullets)
     //   applies an operation to every iterator in such subrange. We may as
     //   well call <code>on_subranges</code> the "worker".
     // - Grainsize: minimum number of iterators (in this case representing
@@ -1271,10 +1272,10 @@ namespace Step69
         0, n_locally_relevant);
 
       const auto on_subranges = //
-        [&](const auto i1, const auto i2) {
+        [&](const auto index_begin, const auto index_end) {
           for (const auto row_index :
-               std_cxx20::ranges::iota_view<unsigned int, unsigned int>(*i1,
-                                                                        *i2))
+               std_cxx20::ranges::iota_view<unsigned int, unsigned int>(
+                 *index_begin, *index_end))
             {
               // First column-loop: we compute and store the entries of the
               // matrix norm_matrix and write normalized entries into the
@@ -1810,17 +1811,17 @@ namespace Step69
     // <code>nij_matrix</code> above are used here again.
     //
     // We define again a "worker" function <code>on_subranges</code> that
-    // computes the viscosity $d_{ij}$ for a subrange [i1, i2) of column
-    // indices:
+    // computes the viscosity $d_{ij}$ for a subrange `[*index_begin,
+    // *index_end)` of column indices:
     {
       TimerOutput::Scope scope(computing_timer,
                                "time_stepping - 1 compute d_ij");
 
       const auto on_subranges = //
-        [&](const auto i1, const auto i2) {
+        [&](const auto index_begin, const auto index_end) {
           for (const auto i :
-               std_cxx20::ranges::iota_view<unsigned int, unsigned int>(*i1,
-                                                                        *i2))
+               std_cxx20::ranges::iota_view<unsigned int, unsigned int>(
+                 *index_begin, *index_end))
             {
               const auto U_i = gather(U, i);
 
@@ -1913,12 +1914,12 @@ namespace Step69
       // locally.
 
       const auto on_subranges = //
-        [&](const auto i1, const auto i2) {
+        [&](const auto index_begin, const auto index_end) {
           double tau_max_on_subrange = std::numeric_limits<double>::infinity();
 
           for (const auto i :
-               std_cxx20::ranges::iota_view<unsigned int, unsigned int>(*i1,
-                                                                        *i2))
+               std_cxx20::ranges::iota_view<unsigned int, unsigned int>(
+                 *index_begin, *index_end))
             {
               double d_sum = 0.;
 
@@ -1999,8 +2000,9 @@ namespace Step69
                                "time_stepping - 3 perform update");
 
       const auto on_subranges = //
-        [&](const auto i1, const auto i2) {
-          for (const auto i : boost::make_iterator_range(i1, i2))
+        [&](const auto index_begin, const auto index_end) {
+          for (const auto i :
+               boost::make_iterator_range(index_begin, index_end))
             {
               Assert(i < n_locally_owned, ExcInternalError());
 
@@ -2249,11 +2251,12 @@ namespace Step69
     // global maxima and minima of the gradients.
     {
       const auto on_subranges = //
-        [&](const auto i1, const auto i2) {
+        [&](const auto index_begin, const auto index_end) {
           double r_i_max_on_subrange = 0.;
           double r_i_min_on_subrange = std::numeric_limits<double>::infinity();
 
-          for (const auto i : boost::make_iterator_range(i1, i2))
+          for (const auto i :
+               boost::make_iterator_range(index_begin, index_end))
             {
               Assert(i < n_locally_owned, ExcInternalError());
 
@@ -2333,8 +2336,9 @@ namespace Step69
 
     {
       const auto on_subranges = //
-        [&](const auto i1, const auto i2) {
-          for (const auto i : boost::make_iterator_range(i1, i2))
+        [&](const auto index_begin, const auto index_end) {
+          for (const auto i :
+               boost::make_iterator_range(index_begin, index_end))
             {
               Assert(i < n_locally_owned, ExcInternalError());
 

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.