]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Move functions in anonymous namespace 8522/head
authorRene Gassmoeller <rene.gassmoeller@mailbox.org>
Wed, 14 Aug 2019 21:12:53 +0000 (14:12 -0700)
committerRene Gassmoeller <rene.gassmoeller@mailbox.org>
Wed, 14 Aug 2019 21:12:53 +0000 (14:12 -0700)
include/deal.II/particles/particle_handler.h
source/particles/particle_handler.cc
source/particles/particle_handler.inst.in

index d7e300d32518fd9486e045790f1bdde4a9625237..cac550d4e0148ebb50a51c9119dfee69c07e076f 100644 (file)
@@ -38,36 +38,6 @@ DEAL_II_NAMESPACE_OPEN
 
 namespace Particles
 {
-  namespace internal
-  {
-    /**
-     * This function packs a vector of particles into a vector
-     * of bytes. If there are properties associated with the
-     * particles they are also serialized. The function does
-     * not perform compression, but is faster
-     * than the corresponding function Utilities::pack().
-     */
-    template <int dim, int spacedim>
-    std::vector<char>
-    pack_particles(std::vector<Particle<dim, spacedim>> &particles);
-
-    /**
-     * This function unpacks a vector of bytes into a vector
-     * of particles. It assumes the vector of bytes was created
-     * with the pack_particles() function above. If the particles
-     * had properties when they were serialized this function
-     * requires an additional argument that is a PropertyPool
-     * object with the correct number of properties per particle
-     * to be able to correctly unpack the particles.
-     */
-    template <int dim, int spacedim>
-    std::vector<Particle<dim, spacedim>>
-    unpack_particles(
-      const boost::iterator_range<std::vector<char>::const_iterator>
-        &           data_range,
-      PropertyPool &property_pool = PropertyPool(0));
-  } // namespace internal
-
   /**
    * This class manages the storage and handling of particles. It provides
    * the data structures necessary to store particles efficiently, accessor
index 27f101d6f6efb729ff9e6c95616726d1a933a43e..491f34e79d32aa5d29d5928352213206989918f8 100644 (file)
@@ -28,7 +28,7 @@ DEAL_II_NAMESPACE_OPEN
 
 namespace Particles
 {
-  namespace internal
+  namespace
   {
     template <int dim, int spacedim>
     std::vector<char>
@@ -73,7 +73,7 @@ namespace Particles
 
       while (data < &(*data_range.end()))
         {
-          particles.push_back(Particle<dim, spacedim>(data, &property_pool));
+          particles.emplace_back(data, &property_pool);
         }
 
       Assert(
@@ -85,7 +85,7 @@ namespace Particles
 
       return particles;
     }
-  } // namespace internal
+  } // namespace
 
   template <int dim, int spacedim>
   ParticleHandler<dim, spacedim>::ParticleHandler()
@@ -1241,7 +1241,7 @@ namespace Particles
           break;
       }
 
-    return internal::pack_particles(stored_particles_on_cell);
+    return pack_particles(stored_particles_on_cell);
   }
 
   template <int dim, int spacedim>
@@ -1254,7 +1254,7 @@ namespace Particles
     // We leave this container non-const to be able to `std::move`
     // its contents directly into the particles multimap later.
     std::vector<Particle<dim, spacedim>> loaded_particles_on_cell =
-      internal::unpack_particles<dim, spacedim>(data_range, *property_pool);
+      unpack_particles<dim, spacedim>(data_range, *property_pool);
 
     // Update the reference to the current property pool for all particles.
     // This was not stored, because they might be transported across process
index 8973ca70e949ce67a40167f0e33aff8ae1f52ac9..022b8568d2513f14ab614812208990fa763784cf 100644 (file)
@@ -19,21 +19,6 @@ for (deal_II_dimension : DIMENSIONS; deal_II_space_dimension : SPACE_DIMENSIONS)
 #if deal_II_dimension <= deal_II_space_dimension
     namespace Particles
     \{
-      namespace internal
-      \{
-        template std::vector<char>
-        pack_particles(
-          std::vector<Particle<deal_II_dimension, deal_II_space_dimension>>
-            &particles);
-
-        template std::vector<
-          Particle<deal_II_dimension, deal_II_space_dimension>>
-        unpack_particles(
-          const boost::iterator_range<std::vector<char>::const_iterator>
-            &           data_range,
-          PropertyPool &property_pool);
-      \}
-
       template class ParticleHandler<deal_II_dimension,
                                      deal_II_space_dimension>;
     \}

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.