Rename a function; also make it run faster.

The new name is dimension independent. The speed is gained because we do not need
to search a list from the top every time, but it is enough to start searching where
we left off previously.

Generalize the data structures for the cells adjacent to an edge.

In 2d, only two cells can be adjacent to an edge, and the current data structures in the
mesh reorientation algorithm reflect this: each edge had 2 slots for adjacent cells, of which
zero, one, or two could be used at any given time.

On the other hand, in 3d, an arbitrary number of cells may be adjacent to an edge. I eventually
want to extend the algorithm re-written in #3166 to the 3d case as well. This patch generalizes
the data structure representing the cells adjacent to an edge by making it generic. The patch
only implements the 2d case, but the 3d case will be easy to add later on, and it makes the 'Edge'
data structure generic.

Merge pull request #3217 from bangerth/fix-test-2

Do not use typename outside templates.

Merge pull request #3216 from bangerth/fix-test

Fix test output.

Do not use typename outside templates.

Fix test output.

Merge pull request #3215 from masterleinad/fix_sign_in_step-15

Fix sign in documentation of step-15

Fix sign in documentation of step-15

Merge pull request #3212 from bangerth/workaround-initializer-lists

Do not use initializer lists in delegating constructors.

Merge pull request #3213 from masterleinad/add_flux_test

Add test for make_flux_sparsity_pattern with periodic bc

Merge pull request #3210 from spco/periodic_sp_fix

Add periodic_neighbor_face_no() call.

Merge pull request #3214 from bangerth/doc-update

List FE_Enriched as a class only enabled with C++11.

List FE_Enriched as a class only enabled with C++11.

Add test for make_flux_sparsity_pattern with periodic bc

Merge pull request #3197 from bangerth/fix-tests

Fix several tests.

Do not use initializer lists in delegating constructors.

This apparently confuses compilers that claim that they can do C++11 but
really can't.

Add periodic_neighbor_face_no() call.

Merge pull request #3208 from tjhei/fe_enriched_cxx14

disable FE_Enriched without cxx14

Merge pull request #3206 from masterleinad/fix_extrapolate_complex

Fix ExtrapolateImplementation for arbitrary VectorTypes

Remove more update flags that are not needed.

Merge pull request #3205 from bangerth/fix-error-message-2

Make exception messages more consistent.

Merge pull request #3203 from bangerth/doc-update-9

Add a test.

disable tests without cxx14

disable FE_enriched without cxx14

Merge pull request #3186 from tjhei/no_set_bdry_id_internal

disallow set_boundary_id() on internal faces

Fix ExtrapolateImplementation for arbitrary VectorTypes

Merge pull request #3204 from tjhei/fix_torus_test_output

fix grid/torus_01 test output

Merge pull request #3202 from masterleinad/silence_enriched_warning

Silence warning in FE_Enriched

Silence warning in FE_Enriched

Add a test that verifies that GridReordering fixes edge orientations.

Make exception messages more consistent.

In particular, change the following two things: (i) Say 'Additional information'
instead of 'Additional Information' because the rest of the text is also
capitalized in the common way. (ii) Indent the actual (additional) error text
by four characters, just like the other additional pieces of information.

The result is that errors now look like this:

--------------------------------------------------------
An error occurred in line <710> of file </home/fac/f/bangerth/p/deal.II/1/install/examples/step-6/step-6.cc> in function
    int main()
The violated condition was:
    false
Additional information:
    abcdef

Stacktrace:
-----------
#0  ./step-6: main
--------------------------------------------------------

fix grid/torus_01 test output

disallow set_boundary_id() on internal faces

Merge pull request #3189 from bangerth/fix-error-message

Tighten up error messages.

Merge pull request #3200 from davydden/fe_enriched_tests

add two Asserts in fe_enriched_step-36 unit test

Merge pull request #3195 from bangerth/fix-test-5

Do not use 'typename' outside templates.

minor cleanup of fe_enriched_step-36 unit test

Merge pull request #3198 from bangerth/doc-update

Update the documentation of Manifold::normal_vector().

add two Asserts in fe_enriched_step-36 unit test

Merge pull request #3191 from bangerth/fix-test

Fix output of test.

Merge pull request #3190 from bangerth/fix-dereference

Do not dereference pointers unless we know that they are non-NULL.

Update the documentation of Manifold::normal_vector().

Merge pull request #3196 from masterleinad/mem_64bit

Fix the memory_consumption output for 64bit indices

Merge branches 'fix-test-6', 'fix-test-7' and 'fix-test-8' into fix-tests

Do not use 'typename' outside templates.

Do not use 'typename' outside templates.

Remove unused variables. Do not compute unused data in FEValues.

Do not use 'typename' outside templates.

Do not use 'typename' outside templates.

Fix the test output for 64bit indices as well

Merge pull request #3193 from bangerth/fix-test-3

Do not use 'typename' outside templates.

Merge pull request #3194 from bangerth/fix-test-4

Do not use 'typename' outside templates.

Merge pull request #3192 from bangerth/fix-test-2

Do not use 'typename' outside templates.

Do not use 'typename' outside templates.

Do not use 'typename' outside templates.

Do not use 'typename' outside templates.

Fix output of test.

The memory consumption for Triangulation recently changed with #3163.
I have been seeing the difference for finite elements for longer already,
but don't recall which change broke the test.

Do not dereference pointers unless we know that they are non-NULL.

Tighten up error messages.

It has long annoyed me that when we print exception messages that were created
via ExcMessage, that the information is really duplicative. Example:

--------------------------------------------------------
An error occurred in line <710> of file <.../step-6.cc> in function
    int main()
The violated condition was:
    false
The name and call sequence of the exception was:
    ExcMessage ("abc" "def")
Additional Information:
abcdef

Stacktrace:
-----------
#0  ./step-6: main
--------------------------------------------------------

Here, the text that comes after 'The name and call sequence...'
is duplicative with what comes after 'Additional Information:'.

This is specifically the case for ExcMessage errors because
there *everything* that constructs the 'Additional information'
will actually be also under 'The name and...' part.

This patch cleans this up by omitting the first of these two
parts if the error message was created via ExcMessage.

Merge pull request #3187 from tjhei/remove_warnings

remove warnings

remove warnings

clang warns with.
```
/ssd/deal-git/source/fe/fe_tools_extrapolate.cc:708:20: warning:
reference cannot be bound to dereferenced null pointer in well-defined
C++ code; comparison may be assumed to always evaluate to true
[-Wtautological-undefined-compare]
```

Merge pull request #3179 from tjhei/mg_comments

improve comments in multigrid method

Merge pull request #3185 from tjhei/fix_hyper_cube_slit

Fix hyper cube slit

fix test output

indent

changelog entry

fix colorized hyper_cube_slit

GridGenerator::hyper_cube_slit() was setting the wrong boundary ids when
colorize was set to true.

Merge pull request #3173 from bangerth/updates-to-the-P1NC-element

Update the P1NC element implementation.

update comments in multigrid method

Update the P1NC element implementation.

Specifically, this updates the class documentation for language
and clarity. It also tightens up the code in a number of places.

Merge pull request #3182 from bangerth/fix-tests

Correct ParameterHandler test outputs.

Correct test outputs.

On my desktop, I have not installed numdiff yet, and these tests fail because
there are either differences in whitespace, or there is a trailing colon on
a line. Neither seem to bother numdiff, but they bother regular diff. Fix this
by checking in corrected versions of these output files.

Merge pull request #3177 from bangerth/doc-update

Fix formatting of doc text.

Merge pull request #3178 from masterleinad/fix_style

Fixup all the other ':: '

Merge pull request #2753 from masterleinad/martin_extrapolate

FETools::extrapolate rebased

Fix related tests

Hide parallel implementation and use guards for vtk test output

Make sure to receive messages from the correct round

Add dealii comment blob and split extrapolate into an own file

Entry to changes.h

Fix doxygen documentation

guard implementation by check for p4est

Make C++03 conforming

Add tests

Fix implementation

Restructure fe_tools_interpolate.cc

move p4est wrappers

Fix some includes for rebasing

FETools::extrapolate: add forward declaration

FETools::extrapolate: Working on the extrapolate algorithm on distributed parallel grids

Fixup all the other ':: '

Fix formatting of doc text.

Merge pull request #3176 from bangerth/fix-style-issue

Eliminate an unnecessary space.

Eliminate an unnecessary space.

Merge pull request #3172 from bangerth/doc-update

Update the description of ConstraintMatrix::print().

Merge pull request #3155 from Rombur/trapezoid

Add GridGenerator for a general quadrilateral/general hexahedron.

Add GridGenerator for a general cell.

Merge pull request #3170 from kronbichler/documentation_fe_evaluation

Augment documentation

Merge pull request #3174 from JaeryunYim/master

Add a description of the addition of FE_P1NC

Merge pull request #3119 from kronbichler/master

Restrict the complete defect at once to the coarse level.