From: Wolfgang Bangerth <bangerth@math.tamu.edu>
Date: Tue, 11 Aug 2015 22:19:19 +0000 (-0500)
Subject: Remove fields from Mapping::InternalDataBase.
X-Git-Tag: v8.4.0-rc2~626^2
X-Git-Url: https://gitweb.dealii.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2e0fd2f6275a68c489975e5e86a4da349a03308f;p=dealii.git

Remove fields from Mapping::InternalDataBase.

It turns out that all of the non-is_first_cell/clear_first_cell() member
variables of Mapping::InternalDataBase are only referenced in Mapping
member functions that are unused throughout the rest of the library.
While they were public, they took an internal object as argument, so
were almost certainly not used anywhere by anyone. Consequently,
zap the fields and the functions that use them.
---

diff --git a/include/deal.II/fe/mapping.h b/include/deal.II/fe/mapping.h
index 0f28c9b49e..b15411db4e 100644
--- a/include/deal.II/fe/mapping.h
+++ b/include/deal.II/fe/mapping.h
@@ -416,28 +416,6 @@ public:
      */
     virtual std::size_t memory_consumption () const;
 
-    /**
-     * The positions of the mapped (generalized) support points.
-     */
-    std::vector<Point<spacedim> > support_point_values;
-
-    /*
-     * The Jacobian of the
-     * transformation in the
-     * (generalized) support
-     * points.
-     */
-    std::vector<Tensor<2,spacedim> > support_point_gradients;
-
-    /*
-     * The inverse of the
-     * Jacobian of the
-     * transformation in the
-     * (generalized) support
-     * points.
-     */
-    std::vector<Tensor<2,spacedim> > support_point_inverse_gradients;
-
   private:
     /**
      * Initially set to true, but reset to false when clear_first_cell()
@@ -582,29 +560,6 @@ public:
              const InternalDataBase &internal,
              const MappingType type) const = 0;
 
-  /**
-   * The transformed (generalized) support point.
-   */
-  const Point<spacedim> &support_point_value(
-    const unsigned int index,
-    const typename Mapping<dim,spacedim>::InternalDataBase &internal) const;
-
-  /**
-   * The Jacobian matrix of the transformation in the (generalized) support
-   * point.
-   */
-  const Tensor<2,spacedim> &support_point_gradient(
-    const unsigned int index,
-    const typename Mapping<dim,spacedim>::InternalDataBase &internal) const;
-
-  /**
-   * The inverse Jacobian matrix of the transformation in the (generalized)
-   * support point.
-   */
-  const Tensor<2,spacedim> &support_point_inverse_gradient(
-    const unsigned int index,
-    const typename Mapping<dim,spacedim>::InternalDataBase &internal) const;
-
   /**
    * Return a pointer to a copy of the present object. The caller of this copy
    * then assumes ownership of it.
@@ -908,43 +863,6 @@ Mapping<dim,spacedim>::InternalDataBase::clear_first_cell ()
 }
 
 
-
-template <int dim, int spacedim>
-inline
-const Point<spacedim> &
-Mapping<dim,spacedim>::support_point_value(
-  const unsigned int index,
-  const typename Mapping<dim,spacedim>::InternalDataBase &internal) const
-{
-  AssertIndexRange(index, internal.support_point_values.size());
-  return internal.support_point_values[index];
-}
-
-
-template <int dim, int spacedim>
-inline
-const Tensor<2,spacedim> &
-Mapping<dim,spacedim>::support_point_gradient(
-  const unsigned int index,
-  const typename Mapping<dim,spacedim>::InternalDataBase &internal) const
-{
-  AssertIndexRange(index, internal.support_point_gradients.size());
-  return internal.support_point_gradients[index];
-}
-
-
-template <int dim, int spacedim>
-inline
-const Tensor<2,spacedim> &
-Mapping<dim,spacedim>::support_point_inverse_gradient(
-  const unsigned int index,
-  const typename Mapping<dim,spacedim>::InternalDataBase &internal) const
-{
-  AssertIndexRange(index, internal.support_point_inverse_gradients.size());
-  return internal.support_point_inverse_gradients[index];
-}
-
-
 #endif // DOXYGEN
 
 DEAL_II_NAMESPACE_CLOSE