fix crash in petsc step-40

fix:

An error occurred in line <96> of file </ssd/deal-
git/source/lac/petsc_solver.cc> in function
    void dealii::PETScWrappers::SolverBase::solve(const
dealii::PETScWrappers::MatrixBase&, dealii::PETScWrappers::VectorBase&,
const dealii::PETScWrappers::VectorBase&, const
dealii::PETScWrappers::PreconditionerBase&)
The violated condition was:
    ierr == 0
The name and call sequence of the exception was:
    ExcPETScError(ierr)
Additional Information:
An error with error number 80 occurred while calling a PETSc function

Merge pull request #2110 from bangerth/remove-unnecessary-friend-declaration

Remove a friend declaration that is unnecessary.

Remove a friend declaration that is unnecessary.

It also made no sense. Everything compiles just fine without it.

Merge pull request #2109 from bangerth/further-FEValues-cleanups-39

Remove a comment that's no longer correct.

Remove a comment that's no longer correct.

In fact, remove the entire section since there is a link to an extensive
documentation module just a couple of lines further down.

Merge pull request #2108 from bangerth/fix-changelog

Sort changelog entries into the right categories.

Merge pull request #2093 from drwells/mesh-concept

Mesh concept

Sort changelog entries into the right categories.

Merge GlossMeshAsAContainer and ConceptMeshType.

This moves the discussion completely into concepts.h.

Merge pull request #2107 from bangerth/fix-changelog

Fix grammar of an entry in the changelog.

Prefer 'MeshType' to 'Container' and 'GridClass'.

These things are already described as meshes in the documentation and there is
already a class named Mesh (buried in grid_reordering_internal.h), so the best
name for the concept is MeshType.

Fix grammar of an entry in the changelog.

Merge pull request #2103 from bangerth/further-FEValues-cleanups-36

Remove an unused argument of FiniteElementData.

Remove an unused argument of FiniteElementData.

The value of this argument was unused by this class, but several derived
classes passed something down anyway. Fix this by simply removing the argument.

Remove a default constructor that likely doesn't do very much good.

Merge pull request #2102 from tjhei/fix_test

fix test codim_one/error_estimator_02

fix test codim_one/error_estimator_02

fixes dealii/tests/codim_one/error_estimator_02.cc:75:9: warning: using
integer absolute value function 'abs' when argument is of floating point
type [-Wabsolute-value]
    if (abs(p(1)-1.0)<1e-5)

Merge pull request #2097 from davydden/slepc_tests_no_iterations

do not print-out iteration counts for GHEP SLEPc unit tests

Merge pull request #2092 from tjhei/kelly_codim_one

Kelly codim one

do not print-out iteration counts for GHEP SLEPc unit tests

fix multithreading bug in test

add changelog

fix Neumann part of 1d Kelly estimator

Neumann conditions were evaluated at the wrong point. Also adjust test
(and print coordinates to prove that it is correct now).

Merge pull request #2096 from tjhei/doxygen_remove_duplicate_functs

doxygen: do not export duplicate functions

doxygen: do not export duplicate functions

Since we started handing .cc files to doxygen, every explicit template
declaration of a function inside a .cc would generate another line for
that function in the class documentation (not clickable and before the
correct ones). We fix this by hiding undocumented functions. A quick
test shows, that even with this change, undocumented functions inside
classes defined in the .h will still be exported, because things like
@name groups and @ingroup count as "documentation".

fix 1d Kelly estimator

fix test

Merge pull request #2094 from kronbichler/fix_test_runs

Only run tests if Trilinos is available.

Only run tests if Trilinos is available.

Merge pull request #2091 from drwells/fix-documentation-verbatim-block

Fix documentation verbatim block

remove unused code

fix test formatting

rewrite normal vector computation

Improve formatting in a documentation example.

Prefer code to verbatim environments.

The pair '@code' and '@endcode' will instruct doxygen to perform syntax
highlighting while the verbatim environment does not.

Merge pull request #2090 from drwells/dofhandlertype-doc

Fix some DoFHandlerType documentation.

Merge pull request #2089 from drwells/update-comment-indentation-1

Update comment indentation 1

Fix some DoFHandlerType documentation.

Fill in the rest of a missing comment.

Reindent comments in the documentation: part 2.

Up until 801cfbfc1d deal.II used a style where comments would start at
about the 20th character for visibility. Some comments within the
documentation were not updated at the time.

Merge pull request #2087 from kronbichler/master

Fix compiler warning

WIP kelly fix

Fix compiler warning

Merge pull request #2086 from tjhei/hyper_shell_followup

follow-up on hyper_shell documentation fixes

follow-up on hyper_shell documentation fixes

add failing test

Merge pull request #2085 from kronbichler/fix_msvc_compilation_02

Second attempt to fix MSVC on aligned vector

Second attempt to fix MSVC on aligned vector

Merge pull request #2084 from tjhei/hyper_shell_defaults

default cell numbers for hyper_shell in 3d

Merge pull request #2052 from kronbichler/master

Matrix-free level transfer for geometric multigrid

Add changelog entry

default cell numbers for hyper_shell in 3d

- a default of n_cells=0 will now choose 6 coarse cells in 3d (instead
of throwing an exception)
- improve documentation

Small changes to due comments on code.

Make tests a bit faster

Use reproducible random numbers in tests

Unit tests for matrix-free mg transfer

Framework tests (complete multigrid solver)

Test safety

Implement matrix-free multigrid level transfer for FE_Q/FE_DGQ

Add instantiation

Merge pull request #2083 from tjhei/fix_multigrid_deadlock2

Fix multigrid deadlock part 2

Merge pull request #2082 from ESeNonFossiIo/small_error

swap Range Domain in null_operator

fix marking of MG cells

fix warning

swap

Merge pull request #2080 from kronbichler/fix_msvc_compilation

Fix compilation with MSVC

Fix compilation with MSVC

Merge pull request #2079 from tjhei/mg_par_deadlock

Parallel Multigrid deadlock

setup missing parallel level neighbors

Level ghosts were not set up if neighbor is finer and we have no cells
on that finest level. This causes a situation where
Triangulation::level_ghost_owners is not symmetric causing a deadlock
(and maybe also wrong information down the line). This causes a few test
output changes...

add comments and checks

add failing test

Merge pull request #2078 from drwells/improve-line-continuations

Improve line continuations

Remove some unneeded virtual functions.

These functions immediately call the base class versions of themselves
so there is no need for them to be reimplemented.

Improve ParameterHandler continuation handling.

This commit builds on 23f306dfc5. ParameterHandler now correctly handles
continuation lines in both versions of the read_input function, while before it
only handled continuation lines in one version (the file name as a string
version) before.

Closes #333.

Prefer 'std::getline' to just 'getline'.

Rename 'lineno' to 'current_line_n'.

This matches the convention used elsewhere in the library. In addition,
make it consistently an unsigned int.

Improve some ParameterHandler code samples.

The code samples are now formatted in the same way as the rest of the
library. Additionally, I fixed a memory leak in one example and got rid
of 'void main' in another.

Switch from the old comment format to the new one.

Up until 801cfbfc1d deal.II used a style where comments would start at
about the 20th character for visibility. Some comments within the
documentation were not updated at the time.

Simplify the forward declarations.

ParameterHandler does not need a forward declaration because none of the
Pattern classes use it.

Merge pull request #2077 from kronbichler/fix_matrixfree_uninit_variable

Do not run into an unused block of code

Merge pull request #2076 from angelrca/FECollection_new_constructors_docchanges

FECollection new constructors changes.h

Do not run into an unused block of code

FECollection new constructors changes.h

Merge pull request #2074 from drwells/update-changes.h-for-write_msh-fix

Added a changelog entry for the GridOut fixes.

Added a changelog entry for the GridOut fixes.

Merge pull request #2068 from drwells/gridout-write_msh-fixes

GridOut fixes

Also assign unique numbers to UCD elements.

Same as the previous commit.

Closes #814.

Assign unique numbers to GMSH elements.

GMSH will not properly load information from a .msh file if some
element (geometry element, as in line, face, quadrilateral, hexahedron)
indices are repeated.

Partially addresses issue #814.

Merge pull request #2065 from angelrca/FECollection_new_constructors

hp::FECollection new constructors added

Merge pull request #2073 from tjhei/doc_fixes

Doxygen fixes

fix broken tutorial image links

do not replace ...step-xy inside <img links

doxygen fixes

table layout and image fixes

- fix weird case where doxygen shifts the last cell in a table downward
causing them to be not correctly aligned
- cleanup some image layout
- do not break image tags (this will break various scripts)

Merge pull request #2072 from bangerth/fix-typo

Fix typo.

Fix typo.

Merge pull request #2071 from davydden/mpiexec

allow oevrriding MPIEXEC from an environment varible

allow oevrriding MPIEXEC from an environment varible

Fix two documentation links.

Prefer '\n' to std::endl.

std::endl flushes the buffer and '\n' is converted internally to the
correct line ending.

Fix some typos in grid_out.h.