Use 'order' instead of 'degree' in FE_RannacherTurek.

The existing 'degree' member variable shadows a member of the base class,
but worse it does not have the same semantics: the former pertains to the
order of the element, the latter to the polynomial degree. Fix this.

Merge pull request #3473 from drwells/pvs-precondition-block

Pvs precondition block

Merge pull request #3474 from kronbichler/master

Return vector partitioner by reference

Use different names for function arguments.

PreconditionBlock already has two members named 'permutation' and
'inverse_permutation' so this makes things a bit clearer.

I applied the same naming scheme to a separate function where previously
the variables were called 'p' and 'i'.

Merge pull request #3454 from masterleinad/address_3427

Make casts explicit, initialize all variables

Return vector partitioner by reference

Initialize a member variable of PreconditionBlock.

This is the default value used by the AdditionalData struct.

Merge pull request #3472 from bangerth/rename-variables-2

Rename a few local variables to avoid ambiguity with member variables.

Merge pull request #3471 from bangerth/initialize-variables

Initialize member variables.

Remove unused method arguments.

This is a protected method which is always called with two member
variables with identical names as the function arguments.

This was caught by PVS studio.

Rename a few local variables to avoid ambiguity with member variables.

The `MGLevelGlobalTransfer` class has member variables `copy_indices_*` but in the
`fill_and_communicate_copy_indices()` member function also allocates local variables
of the same name. Rename these to make the distinction clearer.

While there, also initialize a few member variables in the constructor of a local
class.

Initialize member variables.

Merge pull request #3469 from drwells/pvs-dof-info-initialize

Initialize two DoFInfo fields.

Merge pull request #3456 from drwells/pvs-sparsity-repeated-expression

Remove some repeated subexpressions.

Merge pull request #3468 from drwells/pvs-integration-info-initialize

Initialize an IntegrationInfo field.

Initialize two DoFInfo fields.

These are the same values used in the reinit function.

Initialize an IntegrationInfo field.

This number is correctly filled by IntegrationInfo::initialize, but this
way it starts out with a bogus value (instead of an undefined one).

Merge pull request #3462 from bangerth/rename-variables-2

Rename some variables.

Merge pull request #3463 from bangerth/rename-variables-3

Rename some variables.

Merge pull request #3464 from bangerth/rename-functions-and-variables

Rename local variables with names equal to members of the base class.

Merge pull request #3465 from bangerth/clarify-code

Clarify the intent of a piece of code.

Merge pull request #3466 from bangerth/initialize-variable-5

Initialize a member variable of LogStream.

Remove a repeated subexpression.

This was caught by PVS studio.

Prefer SparsityPattern::empty() to ad hoc checks.

Merge pull request #3467 from bangerth/initialize-variable-6

Initialize member variables.

Merge pull request #3459 from bangerth/rename-arguments

Rename function arguments.

Merge pull request #3458 from bangerth/rename-variables

Rename some variables.

Initialize member variables.

Initialize a member variable of LogStream.

The variable really was never initialized, even though some
output depended on it. I suspect that it just happened to always
be initialized to 'false'. Make this explicit.

Clarify the intent of a piece of code.

The code uses integer division, followed by a double precision multiplication.
It is admittedly not immediately obvious what it is supposed to do. Clarify
that we really mean the integer division.

Rename local variables with names equal to members of the base class.

This is certainly confusing.

While there, notice that the functions that *compute* quadrature points
and weights are called 'set_*', but are 'const'. That makes no sense.
In fact, they just compute and return these values, so call them
'get_*' instead.

Rename some variables.

There are variables of the same name in the base class. Avoid the
confusion of having similarly named variables in the local scope.

Rename some variables.

There are variables of the same name in the base class. Avoid the
confusion of having similarly named variables in the local scope.

Merge pull request #3450 from bangerth/fix-warning

No need for double parentheses.

Merge pull request #3446 from bangerth/fix-DataOutRotation-build_one_patch

Rename a function argument.

Merge pull request #3453 from bangerth/initialize-variables-4

Initialize member variables.

Merge pull request #3338 from kalj/check-for-compute-capability

Check for compute capability

Merge pull request #3461 from kalj/patch-1

Use a single if block

Use a single if block

Fix warning from PVS about two if blocks with identical condition. Fixes #3369.

Rename function arguments.

The Nedelec class used 'degree' in a few places, but it was
really the 'order' of the element. The polynomial degree as
used in the base class is 'order+1'.

Rename some variables.

There are variables of the same name in the base class. Avoid the
confusion of having similarly named variables in the local scope.

Merge pull request #3455 from masterleinad/fix_parpack_test

Fix parpack test

Merge pull request #3448 from davydden/matrix_free_Laplace_GMG_unit_test

add another matrix-free laplace GMG unit test

Merge pull request #3449 from drwells/pvs-timer-initialization

Add some missing initializations to Timer.

Fix parpack test

Make casts explicit, initialize all variables

Initialize member variables.

Merge pull request #3452 from bangerth/initialize-variables

Initialize member variables.

Initialize member variables.

Rename a function argument.

DataOutRotation::build_one_patch took a variable named 'patches',
but unlike in #3445, it is not a reference to the class's member
variable of the same name. Rather, it is an array this function
needs to fill in, and nobody else sees it. Consequently,
rename it 'my_patches'.

While there, also update the documentation, which is no longer
in agreement with the implementation.

No need for double parentheses.

do not require Trilinos for matrix_free tests which do not use it

add another matrix-free laplace GMG unit test

Add some missing initializations to Timer.

Some of these were caught by PVS studio.

Merge pull request #3445 from bangerth/dont-duplicate-variable

Let DataOut::build_one_patch use a member variable.

Merge pull request #3444 from class4kayaker/static_fix

Fix FE_QHier static analysis warning

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

Initialize a member variable.

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

Initialize member variables to invalid values.

Let DataOut::build_one_patch use a member variable.

Currently, it takes a reference to a variable as argument, but this variable
is a member variable of the class to which it has access. That's duplicative.
To make things even more confusing, the local variable and the member
are named the same.

Fix this by just not taking the reference as argument, and directly
accessing the member of the class.

Merge pull request #3439 from bangerth/simplify-computation

Simplify an initialization.

Merge pull request #3437 from bangerth/be-careful-with-streams

Be more careful with streams.

Merge pull request #3436 from bangerth/avoid-pointless-condition

Avoid a pointless comparison.

Fix FE_QHier static analysis warning

Fix to #3406

Merge pull request #3442 from Rombur/add-assertthrow

Add a forgotten assert.

Add a forgotten assert.

Merge pull request #3441 from bangerth/initialize-variables-2

Initialize a member variable to invalid values.

Merge pull request #3431 from bangerth/make-variable-names-uniform

Make variable names uniform.

Merge pull request #3432 from bangerth/fix-blockvectorbase-memory_consumption

Fix BlockVectorBase::memory_consumption.

Initialize a member variable to invalid values.

Simplify an initialization.

Be more careful with streams.

In particular, stop also if in.fail() is true, not just if in.eof() is true.
I suspect that it matters little in practice, but who knows.

Avoid a pointless comparison.

The function works in 3d, so checking for 'dim>1' makes no difference.

Initialize member variables to invalid values.

Initialize a member variable.

Merge pull request #3433 from bangerth/fix-DataOut-memory_consumption

Update memory consumption output.

Merge pull request #3394 from drwells/pvs-function-lib-warnings

Silence a PVS studio warning about repeated code.

Merge pull request #3395 from drwells/pvs-timestep-uninitialized-warning

Fix an member initialization PVS studio warning.

Update memory consumption output.

Apparently, #3333 adds 96 bytes to the memory required by DataOut.

Fix BlockVectorBase::memory_consumption.

The existing implementation forgot to add the memory required for
the std::vector of blocks, it only added the memory required for
the blocks themselves. It turns out that fixing this also allows
to simplify the code significantly.

Make variable names uniform.

Specifically, in DataPostprocessor-derived classes in the tutorial programs,
use the names of function arguments that we also use in DataPostprocessor.

Merge pull request #3430 from bangerth/rename-local-variables

Rename local variables.

Rename local variables.

Instead of uh, duh, dduh, use standard names for variables.

Merge pull request #3393 from davydden/mf_mass_n_q_points_1d

add n_q_points_1d template argument to MatrixFreeOperators::MassOperator

Merge pull request #3391 from davydden/mf_base_diag_fixes

fix matrix-free unit tests due to a change to DiagonalMatrix in Base …

Merge pull request #3428 from bangerth/remove-accidental-files

Remove accidentally added files.

Remove accidentally added files.

Fix an member initialization PVS studio warning.

The variable is now initialized with the same logic used in the
'restart' function.

use fe_degree+2 quadrature points in matrix_free/mass_operator_0x

Silence a PVS studio warning about repeated code.

What we had is correct, but this patch should fix the warning (and make
a little less work for the optimizer).

add n_q_points_1d template argument to MatrixFreeOperators::MassOperator

the rationale is support different usage cases. Say, when projecting
a function one would use fe_degree+2. However when projecting the
MatrixFree quadrature data given by Table<>, most often this data
corresponds to fe_degree+1. So while being given a MatrixFree data
object, we use it even though the mass matrix could be slightly
under-integrated.

fix matrix-free unit tests due to a change to DiagonalMatrix in Base class

while here, also use LaplaceOperator and add a few missing typedefs.

improve error output

Merge pull request #3387 from bangerth/rename-variables

Rename local variables.

Merge pull request #3385 from bangerth/initialize-variable-2

Initialize a member variable.

Merge pull request #3384 from bangerth/initialize-variable

Initialize a member variable.

Merge pull request #3390 from bangerth/fix-invalid-sizeof

Fix a nonsensical use of sizeof().

Fix a nonsensical use of sizeof().

Merge pull request #3386 from bangerth/initialize-variable-3

Initialize a member variable.

Rename local variables.

The variables in question have the same name as a function argument, and
are only used in a local block. They may as well be renamed.

Initialize a member variable.

Also add a piece to the documentation of a member function.