Remove some dead code.

As the assertion at the top of the patch shows, 'starting_indices' must only have values
between 0 and N (=sparsity.n_rows()). Consequently, the loop that replaces out-of-bounds
values with numbers::invalid_size_type should never trigger. Further, the 'remove_if'
will also never do anything, and the fact that we accidentally not actually do anything
with its return value (as one should for 'remove_if') doesn't matter.

Merge pull request #3498 from bangerth/initialize-members-8

Initialize member variables.

Merge pull request #3503 from bangerth/fix-memory-test

Adjust memory consumption output.

Merge pull request #3502 from kronbichler/master

Fix Chebyshev preconditioner

Better initial guess for Chebyshev vector

Adjust memory consumption output.

Add missing include file

Merge pull request #3499 from kronbichler/remove_memory_consumption_test

Remove alternative for memory consumption

Remove alternative for memory consumption that does not seem necessary any more

Merge pull request #3493 from bangerth/provide-copy-op

Provide a copy operator.

Merge pull request #3483 from masterleinad/ignore_subdomain

Ignore subdomain_id in MatrixFree::resolve_cell

Initialize member variables.

Also rename one to match current coding practices. Add to the
variable's documentation.

Merge pull request #3489 from kronbichler/remove_warning_parallel_block_vector

Correct LinearAlgebra::distributed::BlockVector::memory_consumption()

Add test

Correct LinearAlgebra::distributed::BlockVector::memory_consumption()

Merge pull request #3492 from bangerth/initialize-members-6

Initialize all member variables.

Merge pull request #3497 from bangerth/fix-previous-commit

Fix indexing in a previous commit.

Fix indexing in a previous commit.

Merge pull request #3496 from bangerth/initialize-members-7

Initialize member variables in class ExceptionBase.

Initialize member variables in class ExceptionBase.

Merge pull request #3491 from bangerth/initialize-members-5

Initialize all member variables.

Merge pull request #3494 from bangerth/fix-thread_management

Provide copy operator and initialize member variables.

Merge pull request #3495 from bangerth/fix-readme

Add periods in a couple of places in the README.

Add periods in a couple of places in the README.

Provide copy operator and initialize member variables.

Ignore subdomain_id in MatrixFree::resolve_cell

Provide a copy operator.

Also initialize all member variables.

Initialize all member variables.

Initialize all member variables.

Merge pull request #3487 from kronbichler/remove_warning_matrix_free

Fix warning in MatrixFree

Merge pull request #3488 from kronbichler/remove_warning_shape_info

Initialize all variables in MatrixFreeFunctions::ShapeInfo

Merge pull request #3486 from kronbichler/remove_warning_fe_eval

Avoid warning about missing copy constructor

Initialize all variables in MatrixFreeFunctions::ShapeInfo

Test copy constructor

Avoid warning about missing copy constructor

Fix warning in MatrixFree

Merge pull request #3485 from bangerth/remove-duplicate-variable

Remove the FE_DGPNonparametric::degree variable.

Remove the FE_DGPNonparametric::degree variable.

It shadows the variable of same name in the base class, and always has the
same value.

Merge pull request #3476 from kronbichler/matrix_free_improvements

Matrix-free operator cleanup

Initialize variables

Merge pull request #3481 from drwells/pvs-block-vector-iterator

Clarify the signature of a copy constructor.

Merge pull request #3480 from drwells/pvs-block-matrix-array-entry

Add (private) BlockMatrixArray::Entry::operator=.

Require Trilinos for test

Make DiagonalMatrix member of MFOps::Base a shared_ptr

Let MatrixFreeOperators::Base fix the ghost part of a vector

Avoid warnings about deprecated class

Avoid using deprecated functions in initialization

Insert option for using Chebyshev as solver

Clarify the signature of a copy constructor.

PVS studio claims that we do not implement a copy constructor for this
iterator. It is likely that (unnecessarily) including the template
arguments here confused PVS studio, so remove them.

Add (private) BlockMatrixArray::Entry::operator=.

This class implements a (destructive) copy constructor more or less so
that one can use it with std::vector::push_back, so it should disable
operator=.

This fixes a warning caught by PVS studio.

Merge pull request #3479 from bangerth/clarify-variable-name

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

Merge pull request #3478 from bangerth/initialize-variables-5

Initialize members of IndexSet::Range.

Merge pull request #3477 from bangerth/add-Exception-copy-constructor

Add a (disabled) copy operator to ExceptionBase.

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.

Initialize members of IndexSet::Range.

Merge pull request #3451 from bangerth/fix-scopes

Fix the scopes of variables.

Add a (disabled) copy operator to ExceptionBase.

Merge pull request #3475 from drwells/gcc4-fix

Fix compilation with some matrix free tests.

Fix the scopes of variables.

The current version has variables in outer scopes that are assigned the addresses
of local variables in inner scopes. These addresses are not actually used in the
outer scope after we leave the inner scope, but it's a bad idea anyway. Move
the declaration of the pointers to the inner scopes where they are actually
used.

Fix compilation with some matrix free tests.

This fixes a compilation error with GCC 4.8:

error: no match for ‘operator=’
(operand types are ‘boost::shared_ptr<dealii::Table<2, dealii::VectorizedArray<double> > >’ and ‘long int’)

      scalar_coefficient = NULL;
                         ^

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.