Apply suggestions from code review

Co-authored-by: David Wells <drwells@email.unc.edu>

diff --git a/doc/news/changes/minor/20221220StefanoZampini-b b/doc/news/changes/minor/20221220StefanoZampini-b
index d49048b7884724f61ddeaf839ac16d72965291fd..3d1e8e361ecdc27a8bec034c9aee5482084f3dc5 100644 (file)
--- a/doc/news/changes/minor/20221220StefanoZampini-b
+++ b/doc/news/changes/minor/20221220StefanoZampini-b
@@ -1,3 +1,3 @@
 New: PETScWrappers:BlockSparseMatrix now is also a PETSc MATNEST type.
 <br>
-(StefanZampini, 2022/12/20)
+(Stefano Zampini, 2022/12/20)

diff --git a/include/deal.II/lac/petsc_block_sparse_matrix.h b/include/deal.II/lac/petsc_block_sparse_matrix.h
index 8bb025586d1bb278810366039ab4504e0e77d135..f62d718b2ab17fbfc64f05be287785c1dc188c56 100644 (file)
--- a/include/deal.II/lac/petsc_block_sparse_matrix.h
+++ b/include/deal.II/lac/petsc_block_sparse_matrix.h
@@ -279,7 +279,7 @@ namespace PETScWrappers
        * particular, it should only be used for read-only operations into the
        * matrix.
        */
-      operator Mat() const;
+      operator const Mat &() const;
 
       /**
        * Return a reference to the underlying PETSc type. It can be used to

diff --git a/source/lac/petsc_parallel_block_sparse_matrix.cc b/source/lac/petsc_parallel_block_sparse_matrix.cc
index d744d72f2b98d565764cdf0692b9aee32f371b97..49911c8dee8bca8cd20575e7e6c92b172c478935 100644 (file)
--- a/source/lac/petsc_parallel_block_sparse_matrix.cc
+++ b/source/lac/petsc_parallel_block_sparse_matrix.cc
@@ -35,7 +35,7 @@ namespace PETScWrappers
     BlockSparseMatrix::~BlockSparseMatrix()
     {
       PetscErrorCode ierr = destroy_matrix(petsc_nest_matrix);
-      AssertThrow(ierr == 0, ExcPETScError(ierr));
+      AssertNothrow(ierr == 0, ExcPETScError(ierr));
     }
 
 #  ifndef DOXYGEN