Test petsc/petsc_ts_05: avoid stack-use-after-return

Fixes:
```
134: ==1438450==ERROR: AddressSanitizer: stack-use-after-return on address 0x7fa4f4d4bfb0 at pc 0x56248d8610c2 bp 0x7ffeebad19b0 sp 0x7ffeebad19a8
134: READ of size 1 at 0x7fa4f4d4bfb0 thread T0
134:     #0 0x56248d8610c1 in ExponentialDecay::ExponentialDecay(double, dealii::PETScWrappers::TimeStepperData const&, bool, bool, int)::'lambda'(double, dealii::[148/1839]
ers::MPI::Vector const&, dealii::PETScWrappers::MPI::Vector const&, double, dealii::PETScWrappers::MatrixBase&, dealii::PETScWrappers::MatrixBase&)::operator()(double, deali
i::PETScWrappers::MPI::Vector const&, dealii::PETScWrappers::MPI::Vector const&, double, dealii::PETScWrappers::MatrixBase&, dealii::PETScWrappers::MatrixBase&) const /srv/t
estsuite/dealii/tests/petsc/petsc_ts_05.cc:91:15
134:     #1 0x7fa538cf812b in std::__1::__function::__value_func<void (double, dealii::PETScWrappers::MPI::Vector const&, dealii::PETScWrappers::MPI::Vector const&, double,
dealii::PETScWrappers::MatrixBase&, dealii::PETScWrappers::MatrixBase&)>::operator()[abi:v160006](double&&, dealii::PETScWrappers::MPI::Vector const&, dealii::PETScWrappers:
:MPI::Vector const&, double&&, dealii::PETScWrappers::MatrixBase&, dealii::PETScWrappers::MatrixBase&) const /usr/include/c++/v1/__functional/function.h:510:16
134:     #2 0x7fa538cf812b in std::__1::function<void (double, dealii::PETScWrappers::MPI::Vector const&, dealii::PETScWrappers::MPI::Vector const&, double, dealii::PETScWra
ppers::MatrixBase&, dealii::PETScWrappers::MatrixBase&)>::operator()(double, dealii::PETScWrappers::MPI::Vector const&, dealii::PETScWrappers::MPI::Vector const&, double, de
alii::PETScWrappers::MatrixBase&, dealii::PETScWrappers::MatrixBase&) const /usr/include/c++/v1/__functional/function.h:1156:12                                              134:     #3 0x7fa538cf812b in int dealii::PETScWrappers::call_and_possibly_capture_ts_exception<std::__1::function<void (double, dealii::PETScWrappers::MPI::Vector const&, d
ealii::PETScWrappers::MPI::Vector const&, double, dealii::PETScWrappers::MatrixBase&, dealii::PETScWrappers::MatrixBase&)>, double&, dealii::PETScWrappers::MPI::Vector&, dea
lii::PETScWrappers::MPI::Vector&, double&, dealii::PETScWrappers::MatrixBase&, dealii::PETScWrappers::MatrixBase&>(std::__1::function<void (double, dealii::PETScWrappers::MP
I::Vector const&, dealii::PETScWrappers::MPI::Vector const&, double, dealii::PETScWrappers::MatrixBase&, dealii::PETScWrappers::MatrixBase&)> const&, std::exception_ptr&, st
d::__1::function<void ()> const&, double&, dealii::PETScWrappers::MPI::Vector&, dealii::PETScWrappers::MPI::Vector&, double&, dealii::PETScWrappers::MatrixBase&, dealii::PET
ScWrappers::MatrixBase&) /srv/testsuite/dealii/include/deal.II/lac/petsc_ts.templates.h:87:9
134:     #4 0x7fa538cf78cc in dealii::PETScWrappers::TimeStepper<dealii::PETScWrappers::MPI::Vector, dealii::PETScWrappers::MatrixBase, dealii::PETScWrappers::MatrixBase>::s
olve(dealii::PETScWrappers::MPI::Vector&)::'lambda'(_p_TS*, double, _p_Vec*, _p_Vec*, double, _p_Mat*, _p_Mat*, void*)::operator()(_p_TS*, double, _p_Vec*, _p_Vec*, double,
_p_Mat*, _p_Mat*, void*) const /srv/testsuite/dealii/include/deal.II/lac/petsc_ts.templates.h:489:26
134:     #5 0x7fa538cf73d7 in dealii::PETScWrappers::TimeStepper<dealii::PETScWrappers::MPI::Vector, dealii::PETScWrappers::MatrixBase, dealii::PETScWrappers::MatrixBase>::s
olve(dealii::PETScWrappers::MPI::Vector&)::'lambda'(_p_TS*, double, _p_Vec*, _p_Vec*, double, _p_Mat*, _p_Mat*, void*)::__invoke(_p_TS*, double, _p_Vec*, _p_Vec*, double, _p
_Mat*, _p_Mat*, void*) /srv/testsuite/dealii/include/deal.II/lac/petsc_ts.templates.h:472:31
134:     #6 0x7fa504afbcac in TSComputeIJacobian (/usr/lib64/libpetsc.so.3.19+0x10fbcac)
134:     #7 0x7fa504aa1672  (/usr/lib64/libpetsc.so.3.19+0x10a1672)
134:     #8 0x7fa504afd307 in SNESTSFormJacobian (/usr/lib64/libpetsc.so.3.19+0x10fd307)
134:     #9 0x7fa504a43e66 in SNESComputeJacobian (/usr/lib64/libpetsc.so.3.19+0x1043e66)
134:     #10 0x7fa504a35e98 in SNESSolve_NEWTONLS (/usr/lib64/libpetsc.so.3.19+0x1035e98)
134:     #11 0x7fa504a4968e in SNESSolve (/usr/lib64/libpetsc.so.3.19+0x104968e)
134:     #12 0x7fa504aa25e4  (/usr/lib64/libpetsc.so.3.19+0x10a25e4)
134:     #13 0x7fa504aa014a  (/usr/lib64/libpetsc.so.3.19+0x10a014a)
134:     #14 0x7fa504b03013 in TSStep (/usr/lib64/libpetsc.so.3.19+0x1103013)
134:     #15 0x7fa504b03e02 in TSSolve (/usr/lib64/libpetsc.so.3.19+0x1103e02)
134:     #16 0x7fa538cd2ccc in dealii::PETScWrappers::TimeStepper<dealii::PETScWrappers::MPI::Vector, dealii::PETScWrappers::MatrixBase, dealii::PETScWrappers::MatrixBase>::
solve(dealii::PETScWrappers::MPI::Vector&) /srv/testsuite/dealii/include/deal.II/lac/petsc_ts.templates.h:880:24
134:     #17 0x56248d85849f in ExponentialDecay::run() /srv/testsuite/dealii/tests/petsc/petsc_ts_05.cc:182:32
134:     #18 0x56248d855ca6 in main /srv/testsuite/dealii/tests/petsc/petsc_ts_05.cc:242:20
134:     #19 0x7fa502a50989  (/usr/lib64/libc.so.6+0x23989)
134:     #20 0x7fa502a50a44 in __libc_start_main (/usr/lib64/libc.so.6+0x23a44)
134:     #21 0x56248d73e2d0 in _start (/srv/temp/build/tests/petsc/petsc_ts_05.debug/petsc_ts_05.debug+0x742d0)
```

Merge pull request #15584 from bangerth/tpetra

Fix missing template argument.

Merge pull request #15577 from bangerth/sol-trans-2

Merge pull request #15570 from bangerth/affine-constraints-7

Merge pull request #15581 from drwells/more-closest-point

Use ReferenceCell::closest_point() in more places.

Merge pull request #15578 from masterleinad/remove_deprecated_extract_boundary_dofs

Merge pull request #15583 from Rombur/typo_step_16

Fix typo in step-16

Merge pull request #15575 from drwells/fevalues-resplit

Remove the FEValues instantiation splits.

Merge pull request #15567 from bangerth/explicit

Make a constructor 'explicit'.

Merge pull request #15564 from tamiko/do_not_abuse_subscriptor_pointer

FESystem: do not abuse Subscriptor* as a fancy void*

Merge pull request #15571 from Rombur/pinned_buffer

Use pinned memory when MPI is not GPU aware

Merge pull request #15553 from drwells/p3-dependencies

P3 dependencies

Fix extract_subvector_to for Tpetra

Merge pull request #15562 from bangerth/parallel

Fix documentation of parallel::accumulate_from_subranges().

Merge pull request #15569 from bangerth/26

Avoid unnecessary reinitialization.

Fix missing template argument.

Fix typo in step-16

Use ReferenceCell::closest_point() in more places.

Avoid preprocessor define.

Remove deprecated DoFTools::extract_boundary_dofs

Add changelog entry.

Add test.

Fix SolutionTransfer to properly compress() vectors.

Remove the FEValues splits.

Now that we have ReadVector this class is a lot less expensive to compile. In
release mode I timed:

- for 9.5: 85s for all six files
- for ReadVector: 59.1s for all six files
- for all files unified: 42s

The maximum RSS is measured (by /usr/bin/time -v) as 2.6 GB in debug mode so we
are still under our 3 GB rule-of-thumb.

Merge pull request #15555 from masterleinad/remove_deprecated_new_thread_functions

Remove deprecated Threads functionalities

Clean up some orientation code in FiniteElement.

Clean up some orientation code.

FE_Simplex_FE_{DG}P: Add a basic convergence rate test.

Merge pull request #15197 from drwells/read-vector

Add an abstract base class ReadVector purely for vector access.

Merge pull request #15544 from bangerth/affine-constraints-3

Simplify constructors/operators of AffineConstraints::ConstraintLine.

Merge pull request #15554 from marcfehling/github-jobs

Merge pull request #15549 from masterleinad/enable_if_supported_werror

Use -Werror again in enable_if_supported

Merge pull request #15556 from tamiko/print_test_info

CMake: print a status message after deal_ii_pickup_tests() invocation

Add a changelog entry.

Make a constructor 'explicit'.

Avoid unnecessary reinitialization.

Use pinned memory when MPI is not GPU aware

FESystem: do not abuse Subscriptor* as a fancy void*

Instead use dynamic_cast to check for cast compatibility at runtime.
Interestengly, we already did in the debug Assert to check that
everything is in order.

Fix documentation of parallel::accumulate_from_subranges().

Merge pull request #15381 from kronbichler/mf_create_inner_faces

MatrixFree: Only set up the inner faces when so requested

Remove deprecated Threads functionalities

Merge pull request #15557 from tamiko/move_cmake_file

CMake: move a file to a better location

Merge pull request #15546 from marcfehling/step-55-3d

Make step-55 work in 3D.

Add changelog entries.

Its both a new feature and a small incompatibility.

Do not take scalars by reference.

Work around a doxygen issue where it can't match declaration and definition.

Merge pull request #15521 from luca-heltai/output_nonnestedMG_lessverbose

Make output of non_nested_transfer tests less verbose

Merge pull request #15551 from masterleinad/remove_deprecated_fe_values_views_outputtype

Remove deprecated FEValuesViews::OutputType

Merge pull request #15526 from tamiko/fix_submit_results_cmake

CMake: fix submit_results.cmake

Capitalize Werror

Co-authored-by: Matthias Maier <tamiko@43-1.org>

Merge pull request #15520 from kronbichler/shorten_tests

Shorten compilation time of two MatrixFree tests

Merge pull request #15529 from masterleinad/remove_deprecated_consensus_algorithms

Remove deprecations in consensus algorithms

Fix some tests by explicitly using make_array_view().

Make VectorTools::compute_mean_value() use ReadVector.

Make VectorTools::integrate_difference() use ReadVector.

Make KellyErrorEstimator use ReadVector.

Make all higher derivative functions of FEValues use ReadVector.

Make FEValues::get_function_gradients() use ReadVector.

Make FEValues::get_function_values() use ReadVector.

Make all vectors inherit from ReadVector.

This gives a common interface to all vectors for use by FEValues.

CMake: move a file to a better location

Merge pull request #15550 from bangerth/19

Remove some stray comment. Fix grammar.

Merge pull request #15548 from bangerth/affine-constraints-6

Merge pull request #15547 from bangerth/affine-constraints-5

Merge pull request #15545 from drwells/shorten-face-vertex-location

Make face_vertex_location_location() a lot shorter.

add a news entry

CMake: deal_ii_pickup_tests(): only print a summary if we actually found something

Tests: print a nice summary after the invocation of the setup_tests macro

Merge pull request #15528 from masterleinad/remove_deprecations_celldatatransferbuffer

CMake: deal_ii_pickup_tests(): print a nice summary

Let us print a nice little summary at the end of the
deal_ii_pickup_tests() macro that summarizes how many test artifacts
have been found.

github-actions:

- max out cores on macos

- set number of jobs via cmake on windows

Simplify constructors/operators of AffineConstraints::ConstraintLine.

Merge pull request #15543 from masterleinad/remove_deprecated_signed_distance

Remove deprecated LevelSet namespace

Merge pull request #15530 from masterleinad/remove_deprecated_interface_values

Remove deprecated functions in fe_interface_values.h

CMake: deal_ii_add_test() count how many tests we are setting up

This sets the variables _number_of_tests _number_of_test_dependencies in
the parent scope from which deal_ii_add_test() was invoked.

Merge pull request #15539 from marcfehling/legacysignal

Remove deprecated LegacySignal.

Remove deprecated FEValuesViews::OutputType

Remove some stray comment. Fix grammar.

Unconditionally try enabling ffp-exception-behavior=strict

Use -Werror again in enable_if_supported

Remove ill-defined (and poorly documented) comparison operators for AffineConstraints::ConstraintLine.

Implement AffineConstraints::ConstraintLine::swap().

Make step-55 work in 3D.

Make face_vertex_location_location() a lot shorter.

We can just call the other functions directly here. We can always switch this
back if we need better performance.

Remove deprecations in consensus algorithms

Remove deprecated functions in fe_interface_values.h

Remove deprecated LevelSet namespace

Merge pull request #15536 from kronbichler/simplify_index_set

Simplify IndexSet::add_index()

Merge pull request #15537 from kronbichler/reduce_header_inclusions2

MatrixFree: Reduce header inclusions in expensive factory files

Merge pull request #15535 from kronbichler/index_set_complexity

Avoid one potential quadratic complexity case in IndexSet::add_ranges

Merge pull request #15516 from kronbichler/simplify_job_supports_mpi

Move job_supports_mpi() to a more central place

make output less verbose in non-nested mg tests

Merge pull request #15489 from bangerth/const

Catch exceptions as const-reference when possible.

MatrixFree: Reduce header inclusions in expensive factory files

Simplify IndexSet::add_index()

Changelog

Test case that would previously run in quadratic complexity

Avoid quadratic complexity in IndexSet::add_ranges

Shorten another matrix-free test

Shorten compilation time of two MatrixFree tests