From: Wolfgang Bangerth <bangerth@colostate.edu>
Date: Sat, 22 Jul 2017 20:31:09 +0000 (-0600)
Subject: Explicitly =delete ConstraintMatrix::operator=().
X-Git-Tag: v9.0.0-rc1~1388^2
X-Git-Url: https://gitweb.dealii.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1ce2a4992a29d143fc3a126b1afdf9d439f082b9;p=dealii.git

Explicitly =delete ConstraintMatrix::operator=().

Currently, it was just made 'private' and not implemented.
---

diff --git a/include/deal.II/lac/constraint_matrix.h b/include/deal.II/lac/constraint_matrix.h
index f720bd510c..adb33df06f 100644
--- a/include/deal.II/lac/constraint_matrix.h
+++ b/include/deal.II/lac/constraint_matrix.h
@@ -197,6 +197,17 @@ public:
    */
   ConstraintMatrix (ConstraintMatrix &&constraint_matrix) = default;
 
+  /**
+   * Copy operator. Like for many other large objects, this operator
+   * is deleted to avoid its inadvertent use in places such as
+   * accidentally declaring a @p ConstraintMatrix object as a
+   * function argument by value, rather than by reference.
+   *
+   * However, you can use the copy_from() function to explicitly
+   * copy ConstraintMatrix objects.
+   */
+  ConstraintMatrix &operator= (const ConstraintMatrix &) = delete;
+
   /**
    * Move assignment operator
    */
@@ -1431,12 +1442,6 @@ private:
                         const Vector<LocalType>              &local_vector,
                         const std::vector<size_type>         &local_dof_indices,
                         const FullMatrix<LocalType>          &local_matrix) const;
-
-  /**
-   * The assignment operator is not implemented for performance reasons. You
-   * can clear() or reinit() and merge() manually if needed.
-   */
-  ConstraintMatrix &operator= (const ConstraintMatrix &other);
 };