]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Remove storing data we end up not using. 1130/head
authorWolfgang Bangerth <bangerth@math.tamu.edu>
Mon, 13 Jul 2015 21:07:29 +0000 (16:07 -0500)
committerWolfgang Bangerth <bangerth@math.tamu.edu>
Mon, 13 Jul 2015 21:07:29 +0000 (16:07 -0500)
We passed a data element around and stored it in a local class, but
ended up not using it after all. This is likely a leftover from a
previous iteration in the code. In any case, removing the member
variable turns out to compile and test fine, so we don't need to
pass this data to the constructor either.

source/distributed/tria.cc

index 033d9de0fcbee413189ea493a5002556a7f235b6..f9fb9dc4e3d4ad5ebbc0389fd0f9716d4f5f51cb 100644 (file)
@@ -1652,8 +1652,7 @@ namespace
   public:
     RefineAndCoarsenList (const Triangulation<dim,spacedim> &triangulation,
                           const std::vector<types::global_dof_index> &p4est_tree_to_coarse_cell_permutation,
-                          const types::subdomain_id                   my_subdomain,
-                          typename internal::p4est::types<dim>::forest &forest);
+                          const types::subdomain_id                   my_subdomain);
 
     /**
      * A callback function that we pass to the p4est data structures when a
@@ -1688,8 +1687,6 @@ namespace
     std::vector<typename internal::p4est::types<dim>::quadrant> coarsen_list;
     typename std::vector<typename internal::p4est::types<dim>::quadrant>::const_iterator current_coarsen_pointer;
 
-    typename internal::p4est::types<dim>::forest forest;
-
     void build_lists (const typename Triangulation<dim,spacedim>::cell_iterator &cell,
                       const typename internal::p4est::types<dim>::quadrant &p4est_cell,
                       const unsigned int myid);
@@ -1713,10 +1710,7 @@ namespace
   RefineAndCoarsenList<dim,spacedim>::
   RefineAndCoarsenList (const Triangulation<dim,spacedim>            &triangulation,
                         const std::vector<types::global_dof_index>   &p4est_tree_to_coarse_cell_permutation,
-                        const types::subdomain_id                    my_subdomain,
-                        typename internal::p4est::types<dim>::forest &forest)
-    :
-    forest(forest)
+                        const types::subdomain_id                    my_subdomain)
   {
     // count how many flags are set and allocate that much memory
     unsigned int n_refine_flags  = 0,
@@ -3384,8 +3378,7 @@ namespace parallel
       RefineAndCoarsenList<dim,spacedim>
       refine_and_coarsen_list (*this,
                                p4est_tree_to_coarse_cell_permutation,
-                               my_subdomain,
-                               *parallel_forest);
+                               my_subdomain);
 
       // copy refine and coarsen flags into p4est and execute the refinement
       // and coarsening. this uses the refine_and_coarsen_list just built,

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.