]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Mark some functions as static.
authorDavid Wells <wellsd2@rpi.edu>
Tue, 3 Jan 2017 17:03:32 +0000 (12:03 -0500)
committerDavid Wells <wellsd2@rpi.edu>
Wed, 4 Jan 2017 00:54:48 +0000 (19:54 -0500)
This was caught by cppcheck: these methods don't depend on state and may
as well be static.

include/deal.II/base/index_set.h
include/deal.II/base/quadrature_lib.h
source/base/quadrature_lib.cc

index 79f66cac53f9b7e0c2a4d608dae36f64f68d9bec..6a2acf99ec5cf39ec7719b5af95e72927573a921 100644 (file)
@@ -804,7 +804,7 @@ private:
               (range_1.end == range_2.end));
     }
 
-    std::size_t memory_consumption () const
+    static std::size_t memory_consumption ()
     {
       return sizeof(Range);
     }
index d9c879223d4824911637757bfb73d88f27c64e1b..38b25cfd7ff6d2b13dc1f898aa9988e7d2d622e2 100644 (file)
@@ -121,7 +121,7 @@ protected:
    * Evaluate the Gamma function $ \Gamma(n) = (n-1)! $.
    * @param n  point of evaluation (integer).
    */
-  long double gamma(const unsigned int n) const;
+  static long double gamma(const unsigned int n);
 };
 
 
@@ -228,14 +228,16 @@ private:
   /**
    * Compute the points of the quadrature formula.
    */
+  static
   std::vector<double>
-  get_quadrature_points(const unsigned int n) const;
+  get_quadrature_points(const unsigned int n);
 
   /**
    * Compute the weights of the quadrature formula.
    */
+  static
   std::vector<double>
-  get_quadrature_weights(const unsigned int n) const;
+  get_quadrature_weights(const unsigned int n);
 };
 
 
@@ -666,10 +668,10 @@ long double QGaussLobatto<1>::
 JacobiP(const long double, const int, const int, const unsigned int) const;
 template <>
 long double
-QGaussLobatto<1>::gamma(const unsigned int n) const;
+QGaussLobatto<1>::gamma(const unsigned int n);
 
-template <> std::vector<double> QGaussLog<1>::get_quadrature_points(const unsigned int) const;
-template <> std::vector<double> QGaussLog<1>::get_quadrature_weights(const unsigned int) const;
+template <> std::vector<double> QGaussLog<1>::get_quadrature_points(const unsigned int);
+template <> std::vector<double> QGaussLog<1>::get_quadrature_weights(const unsigned int);
 
 template <> QMidpoint<1>::QMidpoint ();
 template <> QTrapez<1>::QTrapez ();
index 8ee0ac765209f88cd0ceec6c0d5c1e5eb78344fb..19d292f61e7d58daed0f87179cd434a49d296e85 100644 (file)
@@ -321,7 +321,7 @@ long double QGaussLobatto<1>::JacobiP(const long double x,
 
 
 template <>
-long double QGaussLobatto<1>::gamma(const unsigned int n) const
+long double QGaussLobatto<1>::gamma(const unsigned int n)
 {
   long double result = n - 1;
   for (int i=n-2; i>1; --i)
@@ -432,7 +432,7 @@ QGaussLog<1>::QGaussLog(const unsigned int n,
 
 template <>
 std::vector<double>
-QGaussLog<1>::get_quadrature_points(const unsigned int n) const
+QGaussLog<1>::get_quadrature_points(const unsigned int n)
 {
   std::vector<double> q_points(n);
 
@@ -565,7 +565,7 @@ QGaussLog<1>::get_quadrature_points(const unsigned int n) const
 
 template <>
 std::vector<double>
-QGaussLog<1>::get_quadrature_weights(const unsigned int n) const
+QGaussLog<1>::get_quadrature_weights(const unsigned int n)
 {
   std::vector<double> quadrature_weights(n);
 

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.