Remove an unnecessary copy constructor.

PVS studio rightfully complains that we should define operator= whenever
we explicitly define a copy constructor. However, since our copy
constructor does nothing special it is in fact equivalent to the default
copy constructor, so we can just get get rid of it and let the compiler
generate the correct copying code.

diff --git a/include/deal.II/lac/petsc_matrix_base.h b/include/deal.II/lac/petsc_matrix_base.h
index 4d8ca762197e2016fe5980182b894895dc4e4109..0dd365bfff22683643c6f36ec39897a7e26ed3fc 100644 (file)
--- a/include/deal.II/lac/petsc_matrix_base.h
+++ b/include/deal.II/lac/petsc_matrix_base.h
@@ -82,11 +82,6 @@ namespace PETScWrappers
                   const size_type   row,
                   const size_type   index);
 
-        /**
-         * Copy constructor.
-         */
-        Accessor (const Accessor &a);
-
         /**
          * Row number of the element represented by this object.
          */
@@ -1000,17 +995,6 @@ namespace PETScWrappers
     }
 
 
-    inline
-    const_iterator::Accessor::
-    Accessor (const Accessor &a)
-      :
-      matrix(a.matrix),
-      a_row(a.a_row),
-      a_index(a.a_index),
-      colnum_cache (a.colnum_cache),
-      value_cache (a.value_cache)
-    {}
-
 
     inline
     const_iterator::Accessor::size_type