From f79e97ccc93c903f9d9cfa01a62c4decfe2d831b Mon Sep 17 00:00:00 2001 From: Wolfgang Bangerth Date: Fri, 22 Sep 2023 09:11:04 -0600 Subject: [PATCH] Deprecate AffineConstraints functions taking only one index set. --- include/deal.II/lac/affine_constraints.h | 30 ++++++++++++++---------- 1 file changed, 18 insertions(+), 12 deletions(-) diff --git a/include/deal.II/lac/affine_constraints.h b/include/deal.II/lac/affine_constraints.h index 8691247e28..04d24c5a47 100644 --- a/include/deal.II/lac/affine_constraints.h +++ b/include/deal.II/lac/affine_constraints.h @@ -583,10 +583,11 @@ public: * which is clearly wasteful and not efficient -- and should be considered * a bug. * - * @note This constructor is equivalent to calling the following one with + * @deprecated This constructor is equivalent to calling the following one with * both of its arguments equal to the index set provided here. This * is not wrong, but inefficient. Use the following constructor instead. */ + DEAL_II_DEPRECATED_EARLY explicit AffineConstraints(const IndexSet &locally_stored_constraints); /** @@ -679,7 +680,10 @@ public: * for which degrees of freedom this object can store constraints. See * the discussion in the documentation of the constructor of this class * that takes a single index set as argument. + * + * @deprecated Use the reinit() function with two index set arguments instead. */ + DEAL_II_DEPRECATED_EARLY void reinit(const IndexSet &locally_stored_constraints); @@ -698,8 +702,10 @@ public: /** * Determines if we can store a constraint for the given @p line_n. This * routine only matters in the distributed case and checks if the IndexSet - * allows storage of this line. Always returns true if not in the - * distributed case. + * provided to the constructor or reinit() function that determines + * which constraints to store, allows storage of the line provides + * as argument to this function. As a consequence, it will always return + * true if not in the distributed case. */ bool can_store_line(const size_type line_n) const; @@ -2138,6 +2144,15 @@ inline AffineConstraints::AffineConstraints() +template +inline AffineConstraints::AffineConstraints( + const IndexSet &locally_stored_constraints) + : AffineConstraints(locally_stored_constraints, + locally_stored_constraints) +{} + + + template inline AffineConstraints::AffineConstraints( const IndexSet &locally_owned_dofs, @@ -2159,15 +2174,6 @@ inline AffineConstraints::AffineConstraints( -template -inline AffineConstraints::AffineConstraints( - const IndexSet &locally_stored_constraints) - : AffineConstraints(locally_stored_constraints, - locally_stored_constraints) -{} - - - template inline AffineConstraints::AffineConstraints( const AffineConstraints &affine_constraints) -- 2.39.5