Reindent step-17.

Add sections in the documentation of step-17 and reorder functions to be more like step-8.

Merge pull request #827 from tjhei/tria_number_cache

remove unnecessary communication in Triangulation::update_number_cache

remove unnecessary communication in Triangulation::update_number_cache

Merge pull request #825 from tamiko/fix_warning

Fix warning introduced with the latest commit

Fix warning introduced with the latest commit

Merge pull request #823 from tamiko/fix_icpc_warnings

Override all interpolate() variants in FE_Q_Hierarchical

Merge pull request #824 from tamiko/fix_clang_warning

Silence a false positive "-Wunneeded-internal-declaration"

Override all interpolate() variants in FE_Q_Hierarchical

Silence a false positive "-Wunneeded-internal-declaration"

Clang inlines ```constraints_and_b_v_are_compatible``` into
```do_project``` and after that emits a wrong warning about the former not
being used. Explicitly annotating ```constraints_and_b_v_are_compatible```
with inline helps.

Merge pull request #821 from tamiko/fix_bundled_boost_warnings

Fix warnings in bundled boost

Merge pull request #822 from tamiko/fix_dangling_else

Fix a forgotten -Wdangling-else warning

Fix a forgotten -Wdangling-else warning

Fix warnings in bundled boost

Merge pull request #820 from tamiko/silence_warning

Remove "register" storage location specifier from bundled tbb

Remove "register" storage location specifier from bundled tbb

In c++98, "register" is a mild hint for the compiler that it might be
beneficial to store the variable in a CPU register. This keyword is
deprecated in C++11. Remove it and trust the compiler to figure this out
for itself.

Merge pull request #818 from bangerth/avoid-warning

Avoid a warning with recently enabled -Wxxx flags.

Avoid a warning with recently enabled -Wxxx flags.

Specifically, we get the following warning:

/home/bangerth/p/deal.II/1/dealii/bundled/tbb41_20130401oss/src/tbb/governor.cpp:64:1: warning: missing initializer for member 'tbb::internal::dynamic_link_descriptor::ptr' [-Wmissing-field-initializers]

Merge pull request #817 from tamiko/bugfix

CMake: Remove -Wno-deprecated

Merge pull request #815 from tamiko/simplify_clang_configuration

CMake: Simplify clang configuration

Merge pull request #811 from tamiko/fix_opencascade_warnings

Fix opencascade warnings

Merge pull request #804 from bangerth/introduce-active_cell_index-2

Introduce active cell index 2

Merge pull request #816 from tamiko/small_cleanup

Remove superfluous flag

Merge pull request #813 from tamiko/fix_dangling_else

Simplify nested if/for clauses to avoid -Wdangling-else warnings

CMake: Remove -Wno-deprecated

-Wno-deprecated has nothing to do with -Wno-deprecated-declarations. Thus,
remove it.

Remove superfluous flag

-Wno-long-long is already unconditionally set.

CMake: Simplify clang configuration

It turns out that clang went a long way since 2011/2012. Remove unneeded
-Wno-* switches.

Merge pull request #812 from luca-heltai/fix-pr-mapping-fe

Changed order of arguments in constructor of MappingFEField

Changed the order of the arguments of the constructor of MappingFEField and tests

Simplify nested if/for clauses to avoid -Wdangling-else warnings

Merge pull request #772 from luca-heltai/fix-mapping-fe

Fix MappingFEField

Fixed usage in the documentation of mapping_fe_field.h

fixed documentation of members in mapping_fe_field.cc and added a reference in vector_tools

Fixed initialization issue.

Added get_position_vector to VectorTools

Removed references to update_euler_vector from tets.

Renamed MappingFE to MappingFEField

Fixed documentation.

Fixed indentation.

Inverted DH and VECTOR in template declaration.

Renamed get_fe_mask to get_component_mask

Added entry in changes.h

fixed: destructor, useless lines, commented code. Renamed tests

added missing documentation and solved the thread safe issue

eliminated if statement in compute_shapes_virtual

solved all the problems highlighted in https://github.com/dealii/dealii/pull/727

const quadrature points in update_euler_vector_using_triangulation

Added two test with a non trivial component mask (mapping_fe_field_real_to_unit_q1_mask.cc) and fixed if statement in update_euler_vector_using_triangulation

Added assert dimension in update_euler_vector_using_triangulation

start working on else statement in update_euler

added one test with fe_bernstein (still failing)

Added tests from q1 to q5 and from b1 to b5

Fixed doxygen comments.

Added a few comments and fixed a const reference.

fixed inital guess in transform_real_to_unit

added hyperballboundary in test b2_mask

Silence warnings caused by opencascade headers

CMake: Remove broken configuration

Merge pull request #810 from kronbichler/master

Fix assertion regarding TrilinosWrappers::SparsityPattern::nonlocal_graph

Add test for empty processors (communicated by D. Arndt).

Fix assertion

Merge pull request #809 from tamiko/fix_warnings

Silence bogus unused-parameters|variables warnings

Silence bogus unused-parameters|variables warnings

Merge pull request #805 from tamiko/silence

Restructure -Wno switches, silence bogus warnings in external headers

Merge pull request #807 from ian-r-rose/typename_fix

Remove typename which was causing problems with C++03

Remove typename which was causing problems with C++03

CMake: Set up boost specific -Wno-* switches in configure_boost.cmake

*shhh*

Replace a double loop over cells and indices by a single loop and the use of cell->active_cell_index().

Replace a double loop over cells and indices by a single loop and the use of cell->active_cell_index().

Merge pull request #803 from kronbichler/master

Minor cleanup of vmult: Avoid casting unused parameter

Minor cleanup of vmult: Avoid casting unused parameter

Merge pull request #802 from kronbichler/master

Remove unnecessary friend declaration

Remove unnecessary friend declaration

Replace a double loop over cells and indices by a single loop and the use of cell->active_cell_index().

Replace a double loop over cells and indices by a single loop and the use of cell->active_cell_index().

Replace a double loop over cells and indices by a single loop and the use of cell->active_cell_index().

Replace a double loop over cells and indices by a single loop and the use of cell->active_cell_index().

Replace a double loop over cells and indices by a single loop and the use of cell->active_cell_index().

Replace a double loop over cells and indices by a single loop and the use of cell->active_cell_index().

Merge pull request #765 from bangerth/introduce-active_cell_index

Introduce active cell index

Merge pull request #797 from kronbichler/master

Make SymmetricTensor<2,dim> mostly dimension-independent

Improve order of multiplication/addition in perform_double_contraction which is performance sensitive

List change

Test dimension-independence.

Make most parts of SymmetricTensor dimension-independent

Merge pull request #798 from drwells/void-unused-parameters-2

Void unused parameters, part 2

When obvious, do not name unused parameters.

Comment unused parameter names.

Use a (void) cast on unused parameters.

Remove a variable that only appears in an assert.
This is slightly simpler than declaring the variable and then using
the (void) cast.

Merge pull request #792 from kronbichler/master

Increase performance of SparsityTools::reorder_hierarchical

Merge branch 'drwells-void-unused-parameters'

merges #736
closes #736

Fix indenting

Reenable -Qunused-arguments for clang

Summarize changed with unused variable warnings.

Update clang flags.
This obeys the "fat note" convention.

Use the standard convention for error codes.

Get rid of named temporaries for asserts.
This also avoids writing a lot of (void)s.

Use "#ifdef DEBUG" for complex asserts.
This avoids writing a lot of (void)s.

Add a (void) cast to unused values in .cc files.
This should *crosses fingers* get rid of all unused parameter warnings
when compiling the library.

Enable Trilinos warnings in recent versions.
At least for my local copy (11.12.1) GCC does not raise any warnings.

Only enable 'unused' GCC warnings in debug mode.
In debug mode all parameters and variables *should* be used (i.e., the
Assert macro is expanded).

Use the standard (void) to mark unused parameters.

When obvious, do not name unused parameters.
Rather than writing
Triangulation<dim> &/*tria*/
where tria is a redundant name, I simply leave the unused parameter name
blank.

Comment out unused parameters in source files.
These are neither used in debug nor release mode.

Do not name unused parameters when obvious.
Rather than `SparsityPattern &sp`, it is simpler to write
`SparsityPattern &` with no name.

Switch from (void) to commented parameter name.
For some compiler options the (void) was not included in the source,
leading to extra compiler warnings.

Comment unused parameter names.
While deleting unused parameter names is legal, it may be helpful in the
future to see the comment in something like `unsigned int /*level*/`.

Use the (void) trick to silence warnings.

Merge pull request #796 from kronbichler/fix_test

Fix test in 64bit mode

Fix test in 64bit mode

Increase performance of SparsityTools::reorder_hierarchical

Merge pull request #790 from bangerth/dynamic-sparsity-pattern-iterator

Dynamic sparsity pattern iterator

Merge pull request #794 from bangerth/doc-update-20

Provide documentation to the various namespaces in generic_linear_algebra.h.

Provide documentation to the various namespaces in generic_linear_algebra.h.

Add another test, this time for parallel triangulations.

Merge pull request #793 from tjhei/constraint_matrix_copy

ConstraintMatrix copy explicit and disable assignment

Refactor common code into a function of its own.

Relax requirement on a function so as to allow resetting the
active_cell_index field also for unused (not only for active and
non-active) cells.