]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Avoid const return type
authorDaniel Arndt <arndtd@ornl.gov>
Thu, 6 Jul 2023 18:05:10 +0000 (14:05 -0400)
committerDaniel Arndt <arndtd@ornl.gov>
Fri, 7 Jul 2023 11:54:18 +0000 (07:54 -0400)
25 files changed:
include/deal.II/algorithms/any_data.h
include/deal.II/base/job_identifier.h
include/deal.II/base/mpi.h
include/deal.II/base/polynomials_integrated_legendre_sz.h
include/deal.II/base/utilities.h
include/deal.II/distributed/tria_base.h
include/deal.II/fe/fe_enriched.h
include/deal.II/fe/fe_interface_values.h
include/deal.II/fe/fe_values.h
include/deal.II/grid/tria.h
include/deal.II/lac/affine_constraints.h
include/deal.II/lac/affine_constraints.templates.h
include/deal.II/numerics/data_out.h
include/deal.II/particles/particle.h
include/deal.II/particles/particle_accessor.h
source/base/job_identifier.cc
source/base/mpi.cc
source/base/polynomials_integrated_legendre_sz.cc
source/base/utilities.cc
source/distributed/tria_base.cc
source/fe/fe_enriched.cc
source/fe/fe_values.cc
source/grid/tria.cc
source/numerics/data_out.cc
source/particles/particle.cc

index fafba5b08dab9d4233adfd7c563b9aa4aecc02c2..3495852c7b6eae434c115007c74f6974f138e7c7 100644 (file)
@@ -76,7 +76,7 @@ public:
    * use try_read() instead.
    */
   template <typename type>
-  const type
+  type
   entry(const std::string &name) const;
 
   /**
@@ -91,7 +91,7 @@ public:
    * Use read_ptr() instead!
    */
   template <typename type>
-  const type
+  type
   read(const std::string &name) const;
 
   /**
@@ -135,12 +135,12 @@ public:
 
   /// Read-only access to stored data object by index.
   template <typename type>
-  const type
+  type
   entry(const unsigned int i) const;
 
   /// Dedicated read only access.
   template <typename type>
-  const type
+  type
   read(const unsigned int i) const;
 
   /// Dedicated read only access to pointer object.
@@ -240,7 +240,7 @@ AnyData::entry(const unsigned int i)
 
 
 template <typename type>
-inline const type
+inline type
 AnyData::entry(const unsigned int i) const
 {
   AssertIndexRange(i, size());
@@ -254,7 +254,7 @@ AnyData::entry(const unsigned int i) const
 
 
 template <typename type>
-inline const type
+inline type
 AnyData::read(const unsigned int i) const
 {
   AssertIndexRange(i, size());
@@ -358,7 +358,7 @@ AnyData::entry(const std::string &n)
 
 
 template <typename type>
-inline const type
+inline type
 AnyData::entry(const std::string &n) const
 {
   const unsigned int i = find(n);
@@ -370,7 +370,7 @@ AnyData::entry(const std::string &n) const
 
 
 template <typename type>
-inline const type
+inline type
 AnyData::read(const std::string &n) const
 {
   const unsigned int i = find(n);
index 2c583f784a5805b489d9fb799ad0d9a738d6cd17..33b0ea8b3130a5910925df149dde3d4dd41bf044 100644 (file)
@@ -63,7 +63,7 @@ public:
   /**
    * Return the value of <tt>id</tt>.
    */
-  const std::string
+  std::string
   operator()() const;
 
   /**
index b77fd3e7c8ce5444f66b9ecc13c07b11dd38d775..af1e327252ce063ac1f5dff398ae397728f9b461 100644 (file)
@@ -154,7 +154,7 @@ namespace Utilities
      * Return a vector of the ranks (within @p comm_large) of a subset of
      * processes specified by @p comm_small.
      */
-    const std::vector<unsigned int>
+    std::vector<unsigned int>
     mpi_processes_within_communicator(const MPI_Comm comm_large,
                                       const MPI_Comm comm_small);
 
index 0fa756f357df57be2b6781618dbd8cc1b1b85dac..0aa37b22015f22cc401e20f9d922ed8c53b1f2b4 100644 (file)
@@ -63,7 +63,7 @@ private:
   /**
    * Main function to compute the co-efficients of the polynomial at degree p.
    */
-  static const std::vector<double>
+  static std::vector<double>
   get_coefficients(const unsigned int k);
 };
 
index 35a7ea24f6dae28c965bf19f823b96ccfdd2b5ca..28c6ebf92739f2f009d18c7d877bec6720fe66be 100644 (file)
@@ -894,7 +894,7 @@ namespace Utilities
      * </tr>
      * </table>
      */
-    const std::string
+    std::string
     get_current_vectorization_level();
 
     /**
index 3c22c353e88a361628b985496960c5801fdd419b..1920f639940f9fa398dc2b7279b8678b810cec4f 100644 (file)
@@ -203,10 +203,10 @@ namespace parallel
     const std::set<types::subdomain_id> &
     level_ghost_owners() const;
 
-    const std::weak_ptr<const Utilities::MPI::Partitioner>
+    std::weak_ptr<const Utilities::MPI::Partitioner>
     global_active_cell_index_partitioner() const override;
 
-    const std::weak_ptr<const Utilities::MPI::Partitioner>
+    std::weak_ptr<const Utilities::MPI::Partitioner>
     global_level_cell_index_partitioner(
       const unsigned int level) const override;
 
index 92a8ce7c9de965d43c80eb2be95de57540602bd2..a09d4274ef85e122d18d08533d1732bdea53ba24 100644 (file)
@@ -429,7 +429,7 @@ public:
   /**
    * Return enrichment functions
    */
-  const std::vector<std::vector<std::function<const Function<spacedim> *(
+  std::vector<std::vector<std::function<const Function<spacedim> *(
     const typename Triangulation<dim, spacedim>::cell_iterator &)>>>
   get_enrichments() const;
 
index 7a46e44f13a9da5ecc84faab68b6be295091ba2d..e281f20398da6272fbd180a9b6801ff3d10b6b15 100644 (file)
@@ -1554,7 +1554,7 @@ public:
    * The @p cell_index is either 0 or 1 and corresponds to the cell index
    * returned by interface_dof_to_cell_and_dof_index().
    */
-  const typename Triangulation<dim, spacedim>::cell_iterator
+  typename Triangulation<dim, spacedim>::cell_iterator
   get_cell(const unsigned int cell_index) const;
 
   /**
@@ -2037,7 +2037,7 @@ public:
    * The concept of views is explained in the documentation of the namespace
    * FEValuesViews.
    */
-  const FEInterfaceViews::Scalar<dim, spacedim>
+  FEInterfaceViews::Scalar<dim, spacedim>
   operator[](const FEValuesExtractors::Scalar &scalar) const;
 
   /**
@@ -2046,7 +2046,7 @@ public:
    * finite element. The concept of views is explained in the documentation of
    * the namespace FEValuesViews.
    */
-  const FEInterfaceViews::Vector<dim, spacedim>
+  FEInterfaceViews::Vector<dim, spacedim>
   operator[](const FEValuesExtractors::Vector &vector) const;
 
   /**
@@ -2704,7 +2704,7 @@ FEInterfaceValues<dim, spacedim>::get_update_flags() const
 
 
 template <int dim, int spacedim>
-const typename Triangulation<dim, spacedim>::cell_iterator
+typename Triangulation<dim, spacedim>::cell_iterator
 FEInterfaceValues<dim, spacedim>::get_cell(const unsigned int cell_index) const
 {
   return get_fe_face_values(cell_index).get_cell();
@@ -3142,7 +3142,7 @@ FEInterfaceValues<dim, spacedim>::get_average_of_function_hessians(
 
 /*------------ Inline functions: FEInterfaceValues------------*/
 template <int dim, int spacedim>
-inline const FEInterfaceViews::Scalar<dim, spacedim>
+inline FEInterfaceViews::Scalar<dim, spacedim>
 FEInterfaceValues<dim, spacedim>::operator[](
   const FEValuesExtractors::Scalar &scalar) const
 {
@@ -3157,7 +3157,7 @@ FEInterfaceValues<dim, spacedim>::operator[](
 
 
 template <int dim, int spacedim>
-inline const FEInterfaceViews::Vector<dim, spacedim>
+inline FEInterfaceViews::Vector<dim, spacedim>
 FEInterfaceValues<dim, spacedim>::operator[](
   const FEValuesExtractors::Vector &vector) const
 {
index 369b235ca30977b26ac6fd7118eecc882877615c..4e93cb514c3de7e08a6f873ea7bb6473810cdafe 100644 (file)
@@ -3503,7 +3503,7 @@ public:
   /**
    * Return a triangulation iterator to the current cell.
    */
-  const typename Triangulation<dim, spacedim>::cell_iterator
+  typename Triangulation<dim, spacedim>::cell_iterator
   get_cell() const;
 
   /**
index ad35295984b3babf536bba04386fa688925788cd..3566fbf6413a6fb561aff47efe692b0b733f9b3f 100644 (file)
@@ -1635,7 +1635,7 @@ public:
    * level of the triangulation, which is returned by the function
    * CellAccessor::global_active_cell_index().
    */
-  virtual const std::weak_ptr<const Utilities::MPI::Partitioner>
+  virtual std::weak_ptr<const Utilities::MPI::Partitioner>
   global_active_cell_index_partitioner() const;
 
   /**
@@ -1643,7 +1643,7 @@ public:
    * level of the triangulation, which is returned by the function
    * CellAccessor::global_level_cell_index().
    */
-  virtual const std::weak_ptr<const Utilities::MPI::Partitioner>
+  virtual std::weak_ptr<const Utilities::MPI::Partitioner>
   global_level_cell_index_partitioner(const unsigned int level) const;
 
   /**
index 874e29d9c71dc36456bbaea4902eb82f4afe6811..49c1b6425999f6113404333d44f3d38b8cb7f1e8 100644 (file)
@@ -1727,7 +1727,7 @@ public:
    * @return A range object for the half open range <code>[this->begin(),
    * this->end())</code> of line entries.
    */
-  const LineRange
+  LineRange
   get_lines() const;
 
   /**
index 7c78c97d08f2947439450d679cd616dd0933b5f4..88ef9570659467b07e3a0c14eaf389ae857c6bee 100644 (file)
@@ -75,7 +75,7 @@ AffineConstraints<number>::ConstraintLine::memory_consumption() const
 
 
 template <typename number>
-const typename AffineConstraints<number>::LineRange
+typename AffineConstraints<number>::LineRange
 AffineConstraints<number>::get_lines() const
 {
   return boost::make_iterator_range(lines.begin(), lines.end());
index 66e32256f6e21d8b2734271f26d806bb77bab8a4..942d5f9987d6454118a34ea256c21e25ec417601 100644 (file)
@@ -413,7 +413,7 @@ public:
    * Return the two function objects that are in use for determining the first
    * and the next cell as set by set_cell_selection().
    */
-  const std::pair<FirstCellFunctionType, NextCellFunctionType>
+  std::pair<FirstCellFunctionType, NextCellFunctionType>
   get_cell_selection() const;
 
 private:
index e4580047f12e60b8574c9a5e38b77f0603466028..56110e7857c7f44ccef817ac5095def397fae823 100644 (file)
@@ -376,7 +376,7 @@ namespace Particles
      *
      * @return An ArrayView of the properties of this particle.
      */
-    const ArrayView<double>
+    ArrayView<double>
     get_properties();
 
     /**
@@ -385,7 +385,7 @@ namespace Particles
      *
      * @return An ArrayView of the properties of this particle.
      */
-    const ArrayView<const double>
+    ArrayView<const double>
     get_properties() const;
 
     /**
@@ -624,7 +624,7 @@ namespace Particles
 
 
   template <int dim, int spacedim>
-  inline const ArrayView<const double>
+  inline ArrayView<const double>
   Particle<dim, spacedim>::get_properties() const
   {
     if (has_properties() == false)
index a8d41b7da107eaf000f325f5ddff2ad8ca05e8c4..941a04a7d7faed6932a2c44714392706ccb7f387 100644 (file)
@@ -302,7 +302,7 @@ namespace Particles
      *
      * @return An ArrayView of the properties of this particle.
      */
-    const ArrayView<double>
+    ArrayView<double>
     get_properties();
 
     /**
@@ -310,7 +310,7 @@ namespace Particles
      *
      * @return An ArrayView of the properties of this particle.
      */
-    const ArrayView<const double>
+    ArrayView<const double>
     get_properties() const;
 
     /**
@@ -784,7 +784,7 @@ namespace Particles
 
 
   template <int dim, int spacedim>
-  inline const ArrayView<const double>
+  inline ArrayView<const double>
   ParticleAccessor<dim, spacedim>::get_properties() const
   {
     Assert(state() == IteratorState::valid, ExcInternalError());
@@ -833,7 +833,7 @@ namespace Particles
 
 
   template <int dim, int spacedim>
-  inline const ArrayView<double>
+  inline ArrayView<double>
   ParticleAccessor<dim, spacedim>::get_properties()
   {
     Assert(state() == IteratorState::valid, ExcInternalError());
index 87e7771a77a69dc4bd2acca4dd1bb4d8603e4e17..cdd287439e490fdaaa7d144ff537c83452df7d34 100644 (file)
@@ -50,7 +50,7 @@ JobIdentifier::JobIdentifier()
 }
 
 
-const std::string
+std::string
 JobIdentifier::operator()() const
 {
   return id;
index 30736e44b3b31b215c282bbecfde47ede69a364d..909590a782bf24d8590e4a7df610474a77eec90e 100644 (file)
@@ -177,7 +177,7 @@ namespace Utilities
 
 
 
-    const std::vector<unsigned int>
+    std::vector<unsigned int>
     mpi_processes_within_communicator(const MPI_Comm comm_large,
                                       const MPI_Comm comm_small)
     {
@@ -677,7 +677,7 @@ namespace Utilities
 
 
 
-    const std::vector<unsigned int>
+    std::vector<unsigned int>
     mpi_processes_within_communicator(const MPI_Comm, const MPI_Comm)
     {
       return std::vector<unsigned int>{0};
index ef370bc5caa13ee329b50f2c8754b8c676aeac19..a54460e4aebe5b01d7265285e86c254a17e8592d 100644 (file)
@@ -24,7 +24,7 @@ IntegratedLegendreSZ::IntegratedLegendreSZ(const unsigned int k)
 
 
 
-const std::vector<double>
+std::vector<double>
 IntegratedLegendreSZ::get_coefficients(const unsigned int k)
 {
   std::vector<double> coefficients(k + 1);
index 6f0b287b57f83ba9bf96b8f2bfbad1a285e33b3a..80874182bb6e2cc2c208825a36c65205dc43d2c9 100644 (file)
@@ -935,7 +935,7 @@ namespace Utilities
 
 #endif
 
-    const std::string
+    std::string
     get_current_vectorization_level()
     {
       switch (DEAL_II_VECTORIZATION_WIDTH_IN_BITS)
index 8559b1498114dc1e25f8720542386dc9bcd652c8..72a318013e95a302f8d7cd2a0e1998e34ccc6e97 100644 (file)
@@ -634,7 +634,7 @@ namespace parallel
 
   template <int dim, int spacedim>
   DEAL_II_CXX20_REQUIRES((concepts::is_valid_dim_spacedim<dim, spacedim>))
-  const std::weak_ptr<const Utilities::MPI::Partitioner> TriangulationBase<
+  std::weak_ptr<const Utilities::MPI::Partitioner> TriangulationBase<
     dim,
     spacedim>::global_active_cell_index_partitioner() const
   {
@@ -645,10 +645,9 @@ namespace parallel
 
   template <int dim, int spacedim>
   DEAL_II_CXX20_REQUIRES((concepts::is_valid_dim_spacedim<dim, spacedim>))
-  const std::weak_ptr<const Utilities::MPI::Partitioner> TriangulationBase<
-    dim,
-    spacedim>::global_level_cell_index_partitioner(const unsigned int level)
-    const
+  std::weak_ptr<const Utilities::MPI::Partitioner> TriangulationBase<dim,
+                                                                     spacedim>::
+    global_level_cell_index_partitioner(const unsigned int level) const
   {
     Assert(this->is_multilevel_hierarchy_constructed(), ExcNotImplemented());
     AssertIndexRange(level, this->n_global_levels());
index f0ecc595ab1f1c63868c5a18e474e9590a7440a8..c8fe09fc022c4dd4a936e1ec9a410879a567a18f 100644 (file)
@@ -258,7 +258,7 @@ FE_Enriched<dim, spacedim>::FE_Enriched(
 
 
 template <int dim, int spacedim>
-const std::vector<std::vector<std::function<const Function<spacedim> *(
+std::vector<std::vector<std::function<const Function<spacedim> *(
   const typename Triangulation<dim, spacedim>::cell_iterator &)>>>
 FE_Enriched<dim, spacedim>::get_enrichments() const
 {
index 62f3fb96c853a3fff9fbd2c2b01cb4ecb9040c3d..325932363ee7ee19cce74188486c3bd71ed543d7 100644 (file)
@@ -3765,7 +3765,7 @@ FEValuesBase<dim, spacedim>::get_function_third_derivatives(
 
 
 template <int dim, int spacedim>
-const typename Triangulation<dim, spacedim>::cell_iterator
+typename Triangulation<dim, spacedim>::cell_iterator
 FEValuesBase<dim, spacedim>::get_cell() const
 {
   return present_cell;
index 9b8cdf61acd86599f7cb0823428b6529921444a9..e512e95cc9ce592a42695bcc5aeaa6882b976b16 100644 (file)
@@ -11198,9 +11198,8 @@ MPI_Comm Triangulation<dim, spacedim>::get_communicator() const
 
 template <int dim, int spacedim>
 DEAL_II_CXX20_REQUIRES((concepts::is_valid_dim_spacedim<dim, spacedim>))
-const std::weak_ptr<const Utilities::MPI::Partitioner> Triangulation<
-  dim,
-  spacedim>::global_active_cell_index_partitioner() const
+std::weak_ptr<const Utilities::MPI::Partitioner> Triangulation<dim, spacedim>::
+  global_active_cell_index_partitioner() const
 {
   return number_cache.active_cell_index_partitioner;
 }
@@ -11209,9 +11208,8 @@ const std::weak_ptr<const Utilities::MPI::Partitioner> Triangulation<
 
 template <int dim, int spacedim>
 DEAL_II_CXX20_REQUIRES((concepts::is_valid_dim_spacedim<dim, spacedim>))
-const std::weak_ptr<const Utilities::MPI::Partitioner> Triangulation<
-  dim,
-  spacedim>::global_level_cell_index_partitioner(const unsigned int level) const
+std::weak_ptr<const Utilities::MPI::Partitioner> Triangulation<dim, spacedim>::
+  global_level_cell_index_partitioner(const unsigned int level) const
 {
   AssertIndexRange(level, this->n_levels());
 
index 4980361119bddcdb5bb3259684c9f2c81b51ad65..43b062ee61eb9c72b5981f63a27cdbfef25d8fdb 100644 (file)
@@ -1325,8 +1325,8 @@ DataOut<dim, spacedim>::set_cell_selection(
 
 
 template <int dim, int spacedim>
-const std::pair<typename DataOut<dim, spacedim>::FirstCellFunctionType,
-                typename DataOut<dim, spacedim>::NextCellFunctionType>
+std::pair<typename DataOut<dim, spacedim>::FirstCellFunctionType,
+          typename DataOut<dim, spacedim>::NextCellFunctionType>
 DataOut<dim, spacedim>::get_cell_selection() const
 {
   return std::make_pair(first_cell_function, next_cell_function);
index 2783a3635880d16bf8349e131a06715374c13197..f574e4dd02a74e6a6b40af026cb824101eb12c89 100644 (file)
@@ -326,7 +326,7 @@ namespace Particles
 
 
   template <int dim, int spacedim>
-  const ArrayView<double>
+  ArrayView<double>
   Particle<dim, spacedim>::get_properties()
   {
     return property_pool->get_properties(property_pool_handle);

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.