Introduce CellAccessor::is_active().

The CellAccessor::active() function did not satisfy the
usual naming scheme of that class in which functions are called
CellAccessor::is_ghost(), CellAccessor::is_locally_owned(), etc. As a
consequence, there is now a new function CellAccessor::is_active(),
with the old function deprecated.

diff --git a/include/deal.II/grid/tria_accessor.h b/include/deal.II/grid/tria_accessor.h
index f961522042b0b185661476d5b1d1d8e6f3c64439..dd51bbdf7531999bcb6f190055a004dd9fbd7690 100644 (file)
--- a/include/deal.II/grid/tria_accessor.h
+++ b/include/deal.II/grid/tria_accessor.h
@@ -3411,16 +3411,33 @@ public:
    */
 
   /**
-   * Test whether the cell has children (this is the criterion for activity of
-   * a cell).
+   * Test that the cell has no children (this is the criterion for whether a
+   * cell is called "active").
    *
    * See the
    * @ref GlossActive "glossary"
    * for more information.
+   *
+   * @deprecated This function is deprecated. Use the is_active()
+   *   function instead, which satisfies the naming scheme of other
+   *   functions inquiring about yes/no properties of cells (e.g.,
+   *   is_ghost(), is_locally_owned(), etc.).
    */
+  DEAL_II_DEPRECATED
   bool
   active() const;
 
+  /**
+   * Test that the cell has no children (this is the criterion for whether a
+   * cell is called "active").
+   *
+   * See the
+   * @ref GlossActive "glossary"
+   * for more information.
+   */
+  bool
+  is_active() const;
+
   /**
    * Return whether this cell is owned by the current processor or is owned by
    * another processor. The function always returns true if applied to an

diff --git a/include/deal.II/grid/tria_accessor.templates.h b/include/deal.II/grid/tria_accessor.templates.h
index 92cdd0cc6e452b86c1488858336f78853593a9d9..65f9afe04ec8257f0b936e524c6eec0ed55806e5 100644 (file)
--- a/include/deal.II/grid/tria_accessor.templates.h
+++ b/include/deal.II/grid/tria_accessor.templates.h
@@ -3581,6 +3581,15 @@ CellAccessor<dim, spacedim>::active() const
 
 
 
+template <int dim, int spacedim>
+inline bool
+CellAccessor<dim, spacedim>::is_active() const
+{
+  return !this->has_children();
+}
+
+
+
 template <int dim, int spacedim>
 inline bool
 CellAccessor<dim, spacedim>::is_locally_owned() const