]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Remove fields from Mapping::InternalDataBase. 1328/head
authorWolfgang Bangerth <bangerth@math.tamu.edu>
Tue, 11 Aug 2015 22:19:19 +0000 (17:19 -0500)
committerWolfgang Bangerth <bangerth@math.tamu.edu>
Tue, 11 Aug 2015 22:19:19 +0000 (17:19 -0500)
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.

include/deal.II/fe/mapping.h

index 0f28c9b49eb179ec8cf90dab6960609b726eefc6..b15411db4e02e1f84eba4c08fafe25d4a306ee70 100644 (file)
@@ -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

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.