From 5b5f968c39c0dc3ce915eb17f485f3cee7b3df5b Mon Sep 17 00:00:00 2001 From: Wolfgang Bangerth Date: Mon, 25 Jul 2016 14:09:54 -0500 Subject: [PATCH] Also update the documentation of hp::MappingCollection::push_back(). --- include/deal.II/hp/mapping_collection.h | 14 ++++++++++---- include/deal.II/hp/q_collection.h | 4 ++++ 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/include/deal.II/hp/mapping_collection.h b/include/deal.II/hp/mapping_collection.h index 2c33579a84..1832da3da2 100644 --- a/include/deal.II/hp/mapping_collection.h +++ b/include/deal.II/hp/mapping_collection.h @@ -74,10 +74,16 @@ namespace hp MappingCollection (const MappingCollection &mapping_collection); /** - * Adds a new mapping to the MappingCollection. The mappings have to be - * added in the order of the active_fe_indices. Thus the reference to the - * mapping object for active_fe_index 0 has to be added first, followed by - * the mapping object for active_fe_index 1. + * Adds a new mapping to the MappingCollection. Generally, you will + * want to use the same order for mappings as for the elements of + * the hp::FECollection object you use. However, the same + * considerations as discussed with the hp::QCollection::push_back() + * function also apply in the current context. + * + * This class creates a copy of the given mapping object, i.e., you can + * do things like push_back(MappingQ(3));. The internal copy + * is later destroyed by this object upon destruction of the entire + * collection. */ void push_back (const Mapping &new_mapping); diff --git a/include/deal.II/hp/q_collection.h b/include/deal.II/hp/q_collection.h index 42983b33f3..9698b34825 100644 --- a/include/deal.II/hp/q_collection.h +++ b/include/deal.II/hp/q_collection.h @@ -80,6 +80,10 @@ namespace hp * hp::QCollection objects; they need not even be of the same size in this * case. * + * The same arguments about the order of elements of collections can, by + * the way, also be made about the elements of hp::MappingCollection + * objects. + * * This class creates a copy of the given quadrature object, i.e., you can * do things like push_back(QGauss(3));. The internal copy * is later destroyed by this object upon destruction of the entire -- 2.39.5