Use PetscErrorCode instead of int.

PETSc typedefs PetscErrorCode to be int (so these are equivalent) but this makes
things a bit clearer.

Merge pull request #3576 from drwells/improve-petsc-checks

Improve the output of PETSc exceptions.

Merge pull request #3615 from bangerth/more-cleanups

More cleanups of GridIn::read_vtk().

More cleanups of GridIn::read_vtk().

Specifically, get rid of a pointless array. Make the assumption
that underlies it explicit by adding assertions.

Merge pull request #3610 from masterleinad/template_create_rhs

VectorType in VectorTools::create_*_hand_side

Merge pull request #3601 from bangerth/cleanups

More GridIn::write_vtk() cleanups.

Consistently check PETSc/SLEPc error codes.

Consistently format PETSc/SLEPc error codes.

Improve the output of PETSc exceptions.

Merge pull request #3611 from masterleinad/test_massoperator

Add another test for MatrixFreeOperators::MassOperator

Merge pull request #3600 from bangerth/add-instantiations

Add a few instantiations.

Merge pull request #3609 from masterleinad/fix_permissions

Fix file permissions in FETools

Add another test for MatrixFreeOperators::MassOperator that fails for insufficient Quadrature

Allow a template VectorType in VectorTools::create_right_hand_side and VectorTools::create_boundary_right_hand_side

Fix file permissions in FETools

Merge pull request #3608 from masterleinad/remove_logname

Remove logname variable in tests

Remove logname variable

Merge pull request #3603 from bangerth/fix-type-error

Fix a type error.

Fix a type error.

More GridIn::write_vtk() cleanups.

Add a few instantiations.

Merge pull request #3598 from bangerth/fix-compilation

Fix compilation.

Fix compilation.

Merge pull request #3592 from tamiko/fix_constraint_matrix

LAC: also allow ConstraintMatrix::distribute for std::complex

Merge pull request #3517 from drwells/improve-mpi-checks

Improve mpi checks

Merge pull request #3595 from bangerth/cleanups

Clean up GridIn::read_vtk() in several ways.

Clean up GridIn::read_vtk() in several ways.

Rename some variables. Also note that we always had
  vertex_indices[i] = i
  quad_indices[i] = i
  line_indices[i] = i
and so can get rid of these maps. They are a left-over from apparently copy-pasting
the initial version of this function from the UNV reader where this did not
have to be the case.

The function has many other problems, but I'm an incrementalist and I choose progress
over waiting for the perfect to happen :-).

add a test

LAC: also allow ConstraintMatrix::distribute for std::complex

Fix a typetrait check in ConstraintMatrix::distribute such that said class
and method can be used with complex valued vector types.

Merge pull request #3591 from bangerth/fix-off-by-one-error

Fix an off-by-one error in GridIn::read_vtk().

Merge pull request #3593 from masterleinad/fix_mapping_test

Fix mapping_q_manifold_01

Merge pull request #3587 from bangerth/add-copy-operator

Fix Trilinos SparsityPattern iterators.

Fix mapping_q_manifold_01

Add a testcase.

Add a changelog entry.

Fix an off-by-one error in GridIn::read_vtk.

Add a changelog entry.

Add tests.

Fix the (entirely broken) function Accessor::visit_present_row().

Remove the copy constryctor of TrilinosWrappers::SparsityPatternIterators::Accessor.

It is the default copy constructor, and by removing it, we need not
worry about the fact that we have no copy assignment operator in this class.

Merge pull request #3589 from bangerth/fix-wrong-mapping-clone

Fix a rather insidious bug in MappingQ.

Add a changelog entry.

Add testcases.

Merge pull request #3588 from bangerth/update-some-comments

Update some comments that were not quite accurate any more.

Ensure that cloning a mapping also copies all internal variables.

MappingQ has a member variable 'use_mapping_q_on_all_cells', but this variable is not
replicated when calling MappingQ::clone(). Fix this.

Update some comments that were not quite accurate any more.

Also just minor cleanups. No functional changes.

Merge pull request #3585 from kalj/mg-fix

Fix array lenghts

Merge pull request #3582 from bangerth/add-comment

Add a reference to step-33.

Merge pull request #3584 from bangerth/improve-error

Improve an error message.

Fix array lenghts

Improve an error message.

Merge pull request #3579 from bangerth/initialize-members-2

Initialize the 'matrix' and 'vector' members in the PETSc wrappers.

Add a reference to step-33.

Merge pull request #3581 from bangerth/make-comment-more-readable

Make comment more readable.

Make comment more readable.

Initialize the 'matrix' and 'vector' members in the PETSc wrappers.

These members are of type 'Mat' and 'Vec', which are typedefed to pointers to otherwise
opaque types. Initialize these members to 'NULL'.

Merge pull request #3575 from bangerth/initialize-member-variable-2

Initialize member variables.

Merge pull request #3577 from masterleinad/initialize_quick_tests

Initialize all variables in quick_tests/step

Initialize all variables in quick_tests/step

Merge pull request #3572 from davydden/pvs/parpack

parpack: initialize member variables

Merge pull request #3574 from bangerth/initialize-member-variable

Initialize a member variable.

Initialize member variables.

Initialize a member variable.

Merge pull request #3573 from drwells/petsc-matrix-accessor-copy-operator

Petsc matrix accessor copy operator

Add a test for copying PETSc matrix iterators.

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.

Merge pull request #3570 from bangerth/fix-typo

Fix comment style.

Merge pull request #3528 from masterleinad/update_postprocess

Update DataPostprocessor in examples

Update DataPostprocessor in examples

Fix comment style.

parpack: initialize member variables

Merge pull request #3571 from bangerth/rename-variables

Rename a local variable.

Rename a local variable.

Do not name it the same as a member variable in the base class.
Fixes #3547.

Check all MPI calls.

This commit adds AssertThrowMPI checks to the return code of every MPI
call in deal.II.

Add a specific exception for MPI failures.

Merge pull request #3524 from bangerth/fix-memory-leak

Fix a memory leak in step-39.

Merge pull request #3526 from kronbichler/fix_test_shapes_q_bubbles

Increase accuracy of test output

Merge pull request #3527 from kronbichler/provide_additional_test_output

Provide additional test output.

Fix a memory leak in step-39.

Also avoid some obscure old-style C functions.

Provide additional test output.

Increase accuracy of test output

Merge pull request #3525 from davydden/fix_warning

fix unused variable warning

fix unused variable warning

Merge pull request #3523 from bangerth/fix-memory-leak-3

Fix a memory leak in de-serializing a SparsityPattern object.

Merge pull request #3522 from bangerth/fix-memory-leaks

Fix several more memory leaks in tests.

Fix a memory leak in de-serializing a SparsityPattern object.

Fix memory leaks tests.

Merge pull request #3521 from bangerth/fix-memory-leaks

Fix a variety of memory leaks in the testsuite.

Merge pull request #3519 from bangerth/fix-memory-leak

Avoid a memory leak.

Fix a variety of memory leaks in the testsuite.

Merge pull request #3518 from drwells/fix-hessian-typos

Fix two typos related to the Hessians.

Avoid a memory leak.

In fact, the test really tries very hard to create a memory leak, and
even verifies that it happens. But this then leads to downstream heartbreak
if you run a memory checker because you get pulled over for it.

Fix the issue by cleaning up the memory leak after we have verified that
the code that ran before really did leak the memory, by keeping a pointer
to the leaked object after all, and deleting it at the end of it all.

Fix two typos related to the Hessians.

Merge pull request #3508 from bangerth/fix-test

Reduce the number of steps for high order mappings.

Merge pull request #3509 from bangerth/fix-test-2

Increase accuracy of output in one test.

Merge pull request #3510 from bangerth/fix-slepc-test

Be more lenient with the exact number of iterations.

Merge pull request #3505 from davydden/split_compilation_of_project_qp

split instantiation of VectorTools::project() for quadrature points data

split instantiation of VectorTools::project() for quadrature points data

Be more lenient with the exact number of iterations.

Increase accuracy of output in one test.

This test outputs numbers such as 1/8=0.125 with just two digits after the decimal
point. Consequently, it depends on round-off and optimization flags if we get 0.12
or 0.13. Because the difference is so large, numdiff reports this as a failure.

Fix this by increasing the accuracy to 3 digits.