Initialize all variables in MatrixFreeFunctions::ShapeInfo

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.

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.