Bugfix: Correctly set up object in move constructor

diff --git a/include/deal.II/lac/petsc_parallel_vector.h b/include/deal.II/lac/petsc_parallel_vector.h
index e8ddd6b31bf22512d5de157db9a315363f633808..ea6137f65d968d2fdb981af08a1b5623c92f330c 100644 (file)
--- a/include/deal.II/lac/petsc_parallel_vector.h
+++ b/include/deal.II/lac/petsc_parallel_vector.h
@@ -188,7 +188,7 @@ namespace PETScWrappers
 #ifdef DEAL_II_WITH_CXX11
       // explicitly declare default variant, such that below move constructor
       // does not dissallow it
-      Vector (const Vector &v) = default;
+      Vector (const Vector &) = default;
 
       /**
        * Move constructor. Creates a new vector by stealing the internal data

diff --git a/source/lac/petsc_parallel_vector.cc b/source/lac/petsc_parallel_vector.cc
index 898b98e23c480954e2fda762ab71c20cdea8d82d..c6d0749f3109fc93d14b838e5b6631da565dccd8 100644 (file)
--- a/source/lac/petsc_parallel_vector.cc
+++ b/source/lac/petsc_parallel_vector.cc
@@ -30,10 +30,8 @@ namespace PETScWrappers
 
     Vector::Vector ()
     {
-      // this is an invalid empty vector, so we
-      // can just as well create a sequential
-      // one to avoid all the overhead incurred
-      // by parallelism
+      // this is an invalid empty vector, so we can just as well create a
+      // sequential one to avoid all the overhead incurred by parallelism
       const int n = 0;
       const int ierr
         = VecCreateSeq (PETSC_COMM_SELF, n, &vector);
@@ -46,6 +44,14 @@ namespace PETScWrappers
 #ifdef DEAL_II_WITH_CXX11
     Vector::Vector (Vector &&v)
     {
+      // this is an invalid empty vector, so we can just as well create a
+      // sequential one to avoid all the overhead incurred by parallelism
+      const int n = 0;
+      const int ierr
+        = VecCreateSeq (PETSC_COMM_SELF, n, &vector);
+      AssertThrow (ierr == 0, ExcPETScError(ierr));
+      ghosted = false;
+
       swap(v);
     }
 #endif