Fixed according to comments

- Added description for the patches member of the data out class
- Added lines in  between the functions
- Added forward declaration of the particle handler in the data out
class
- Removed non-useful headers

diff --git a/include/deal.II/particles/data_out.h b/include/deal.II/particles/data_out.h
index 39374a2544c0cf942025481cad39a54fdc203ac1..3feba4c23cb96f660bc77ea1efa35553895de556 100644 (file)
--- a/include/deal.II/particles/data_out.h
+++ b/include/deal.II/particles/data_out.h
@@ -16,22 +16,20 @@
 #define dealii_particles_data_out_h
 
 #include <deal.II/base/config.h>
-
 #include <deal.II/base/data_out_base.h>
-#include <deal.II/base/mpi.h>
-
-#include <deal.II/particles/particle_handler.h>
 
 #include <string>
 #include <vector>
 
-
 DEAL_II_NAMESPACE_OPEN
 
 namespace Particles
 {
+  template <int dim, int spacedim>
+  class ParticleHandler ;
+
   /**
-   * This class manages the DataOut of a Particle Handler
+   * This class generates graphical output for the particles stored by a ParticleHandler object.
    * From a particle handler, it generates patches which can then be used to
    * write traditional output files. This class currently only supports witing
    * the particle position and their ID and does not allow to write the
@@ -46,20 +44,20 @@ namespace Particles
   {
   public:
     /**
-     *Default constructor for the Particles::DataOut class.
+     * Default constructor for the Particles::DataOut class.
      */
     DataOut() = default;
 
     /**
-     *Default destructor for the Particles::DataOut class.
-     */
+     * Destructor for the Particles::DataOut class.
+    */
     ~DataOut() = default;
 
 
     /**
-     * Build the patches for a given particles handler.
+     * Build the patches for a given particle handler.
      *
-     * @param [in] particle_handler A particle handler for which the patches will be built
+     * @param [in] particle_handler A particle handler for which the patches will be built.
      * A dim=0 patch is built for each particle. The position of the particle is
      * used to build the node position and the ID of the particle is added as a
      * single data element.
@@ -78,13 +76,17 @@ namespace Particles
     get_patches() const override;
 
     /**
-     * Returns the name of the data sets associated with the patches. In the
-     * current implementation the particles only contain the ID
+     * Virtual function through which the names of data sets are obtained from this class
      */
     virtual std::vector<std::string>
     get_dataset_names() const override;
 
   private:
+    /**
+     * This is a list of patches that is created each time build_patches() is
+     * called. These patches are used in the output routines of the base
+     * classes.
+     */
     std::vector<DataOutBase::Patch<0, spacedim>> patches;
 
     /**

diff --git a/source/particles/data_out.cc b/source/particles/data_out.cc
index 44a249a14dc52e330041b0fd666c11f8e71aedd6..36ce34b49acc0ca5d4be555119ea5b47c079401b 100644 (file)
--- a/source/particles/data_out.cc
+++ b/source/particles/data_out.cc
@@ -41,6 +41,8 @@ namespace Particles
       }
   }
 
+
+
   template <int dim, int spacedim>
   const std::vector<DataOutBase::Patch<0, spacedim>> &
   DataOut<dim, spacedim>::get_patches() const
@@ -48,6 +50,8 @@ namespace Particles
     return patches;
   }
 
+
+
   template <int dim, int spacedim>
   std::vector<std::string>
   DataOut<dim, spacedim>::get_dataset_names() const