]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Avoid _n and _m as variable names. 9199/head
authorWolfgang Bangerth <bangerth@colostate.edu>
Mon, 30 Dec 2019 14:15:40 +0000 (07:15 -0700)
committerWolfgang Bangerth <bangerth@colostate.edu>
Mon, 30 Dec 2019 14:15:40 +0000 (07:15 -0700)
In general, we don't use variable names that start with underscores, but here using
one-letter variables can also easily be avoided.

include/deal.II/lac/sparse_direct.h
source/lac/sparse_direct.cc

index c3e4bed3022dbb638b3656375f9ddb977e3a494e..6c3685c046d0931adf8f46c9c88129385cd83c98 100644 (file)
@@ -317,14 +317,15 @@ public:
 
 private:
   /**
-   * The dimension of the range space.
+   * The dimension of the range space, i.e., the number of rows of the matrix.
    */
-  size_type _m;
+  size_type n_rows;
 
   /**
-   * The dimension of the domain space.
+   * The dimension of the domain space, i.e., the number of columns of the
+   * matrix.
    */
-  size_type _n;
+  size_type n_cols;
 
   /**
    * The UMFPACK routines allocate objects in which they store information
index 8f655d2d6bab85b9f72d3a04323d0ac368657884..2592e599d9a959ca2a6c48a0cbcc7ae1be414fdc 100644 (file)
@@ -51,8 +51,8 @@ SparseDirectUMFPACK::initialize(const SparsityPattern &)
 #ifdef DEAL_II_WITH_UMFPACK
 
 SparseDirectUMFPACK::SparseDirectUMFPACK()
-  : _m(0)
-  , _n(0)
+  : n_rows(0)
+  , n_cols(0)
   , symbolic_decomposition(nullptr)
   , numeric_decomposition(nullptr)
   , control(UMFPACK_CONTROL)
@@ -198,8 +198,8 @@ SparseDirectUMFPACK::factorize(const Matrix &matrix)
 
     clear();
 
-  _m = matrix.m();
-  _n = matrix.n();
+  n_rows = matrix.m();
+  n_cols = matrix.n();
 
   const size_type N = matrix.m();
 
@@ -367,8 +367,8 @@ SparseDirectUMFPACK::solve(const Matrix &       matrix,
 
 
 SparseDirectUMFPACK::SparseDirectUMFPACK()
-  : _m(0)
-  , _n(0)
+  : n_rows(0)
+  , n_cols(0)
   , symbolic_decomposition(nullptr)
   , numeric_decomposition(nullptr)
   , control(0)
@@ -484,15 +484,15 @@ SparseDirectUMFPACK::Tvmult(BlockVector<double> &      dst,
 SparseDirectUMFPACK::size_type
 SparseDirectUMFPACK::m() const
 {
-  Assert(_m != 0, ExcNotInitialized());
-  return _m;
+  Assert(n_rows != 0, ExcNotInitialized());
+  return n_rows;
 }
 
 SparseDirectUMFPACK::size_type
 SparseDirectUMFPACK::n() const
 {
-  Assert(_n != 0, ExcNotInitialized());
-  return _n;
+  Assert(n_cols != 0, ExcNotInitialized());
+  return n_cols;
 }
 
 

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.