Fix some #ifdef logic in the Timer class.

This fixes a bug due to rewritting an else statement where the 'else' was in an
'ifdef' and the body of the else statement (one line) was outside of it. This
caused, after a rewrite, the body of the former else statement to be executed
twice.

Before 9ef67983fd the end of this function was:

#ifdef DEAL_II_WITH_MPI
      if (sync_wall_time && Utilities::MPI::job_supports_mpi())
        {
          this->mpi_data
            = Utilities::MPI::min_max_avg (last_lap_time, mpi_communicator);
          last_lap_time = this->mpi_data.max;
          cumulative_wall_time += last_lap_time;
        }
      else
#endif
        cumulative_wall_time += last_lap_time;
    }
  return cumulative_time;

where the 'else' statement was run unconditionally if MPI was not present. The
previous (prior to this patch) version was:

#ifdef DEAL_II_WITH_MPI
      this->mpi_data = Utilities::MPI::min_max_avg (last_lap_time,
                                                    mpi_communicator);
      if (sync_wall_time && Utilities::MPI::job_supports_mpi())
        {
          last_lap_time = this->mpi_data.max;
          last_lap_cpu_time = Utilities::MPI::min_max_avg (last_lap_cpu_time,
                                                           mpi_communicator).max;
        }
      cumulative_wall_time += last_lap_time;
      cumulative_time += last_lap_cpu_time;
      this->mpi_total_data = Utilities::MPI::min_max_avg (cumulative_wall_time,
                                                          mpi_communicator);
#endif
      cumulative_wall_time += last_lap_time;
      cumulative_time += last_lap_cpu_time;
    }
  return cumulative_time;

which will, if MPI is available, double count both the cumulative wall time and
cummulative time.

Merge pull request #4687 from bangerth/simplify-code-2

Simplify some code.

Simplify some code.

If you can write code in one line instead of two, then that's worth it :-)

Merge pull request #4683 from bangerth/fix-hp-serialization

Use the proper way to deal with std::unique_ptrs.

Use the proper way to deal with std::unique_ptrs.

Merge pull request #4681 from masterleinad/fix_array_view_09_release

Fix test output for base/array_view_09 in release mode

Fix test output for base/array_view_09 in release mode

Merge pull request #4670 from drwells/improve-arrayview

Improve arrayview

Merge pull request #4677 from masterleinad/implement_constexpr_maxmin

Fix the last remaining constexpr std::max in patterns.h

Fix the last remaining constexpr std::max in patterns.h

Merge pull request #4675 from masterleinad/implement_constexpr_maxmin

Implement constexpr max and min for DEAL_II_WITH_CXX14=false

Implement constexpr max and min for DEAL_II_WITH_CXX14=false

Merge pull request #4674 from masterleinad/rename_test

Rename test requiring Trilinos

Rename test requiring Trilinos

Add iterator-friendly make_array_view functions.

Merge pull request #4660 from gassmoeller/fix_zero_dim_xdmf

Fix DataOutInterface::create_xdmf_entry for dim==0

Merge pull request #4665 from kronbichler/fix_tensor_product_matrix_test

Relax tolerance in tensor product matrix test with floats.

Merge pull request #4671 from tjhei/doc_stokes_l

improve documentation of Functions::StokesLSingularity

improve documentation of Functions::StokesLSingularity

Merge pull request #4659 from drwells/fix-mapping-q1

Fix mapping q1

Add a simple iterator interface to ArrayView.

Use std::remove_cv instead of boost::remove_cv.

Clean up headers and forward declarations for ArrayView.

Relax tolerance in tensor product matrix test with floats.

Describe the changes in a test and changelog entry.

Merge pull request #4657 from masterleinad/alternative_pattern

Add alternative output for pattern_tools_06/07

Fix confusion about dim and spacedim

Mark some quantities in transform_real_to_unit_cell as long double.

This improves accuracy but incurs a modest (about 20%) slowdown. Using the exact
formula here instead of the Newton scheme (that we use for MappingQGeneric)
results in a 40x speed up anyway, so this is relatively small.

Rearrange the MappingQ1 quadratic equation solver.

This gets around an issue where, for parallelograms, a should be zero: the
previous version did not handle this correctly.

Add alternative output for pattern_tools_06/07

Merge pull request #4655 from nfehn/master

Improvement of initialization of Chebyshev smoother.

entry in /doc/news/changes/minor/ added

Improvement of initialization of Chebyshev smoother.

Make initial guess robust with respect to number of processors by operating on the global index.

Merge pull request #4651 from gassmoeller/fix_zero_dim_output

Fix some minor bugs in #4628

Merge pull request #4653 from kronbichler/fix_unity_build_with_clang_3.5

Fix unity build with clang-3.5.

Fix unity build with clang-3.5.

Merge pull request #4652 from masterleinad/fix_compilation_without_p4est

Fix compilation without p4est

Merge pull request #4650 from masterleinad/fix_warning_geometryinfo

Fix warning about instantiation of GeometryInfo<0>

Fix compilation without p4est

Fix some minor bugs

Fix warning about instantiation of GeometryInfo<0>

Merge pull request #4442 from drwells/manual-unity

Manual unity

Merge pull request #4645 from bangerth/constraint-matrix-copy

Implement a ConstraintMatrix::copy_from() function.

Merge pull request #4648 from drwells/add-p4est-zlib-check

Fail to install p4est if it is not compiled with zlib.

Fail to install p4est if it is not compiled with zlib.

Merge pull request #4649 from simonsticko/disable_warning_immersed_surface_quad

Remove a compiler warning in ImmersedSurfaceQuadrature.

Remove a compiler warning in ImmersedSurfaceQuadrature.

Update the documentation on CMakeLists.txt files.

Add an option for a unity build.

Add another FE namespace fix.

Merge pull request #4632 from luca-heltai/pattern-tools

PatternsTools namespace.

Merge pull request #4305 from simonsticko/immersed_surface_quadrature

Add class ImmersedSurfaceQuadrature

Merge pull request #4646 from drwells/add-default-prm-note

Add a note on generating default parameter files.

Merge pull request #4647 from bangerth/doc-update

Minor doc updates to a couple of classes.

Add a note on generating default parameter files.

A since-removed equivalent to parse_input (see commit 8baf33d316b) took,
as an optional argument argument, a boolean that specified whether or
not parse_input should write a default parameter file if one is not
found. This commit summarizes that old documentation for users who want
the old behavior.

Moved changelog to major.

Specialise Bool pattern.

Fixed naming of is_stl_container and is_stl_map

change log

Added add_parameter function and tests.

Moved pattern_tools.h inside patterns.h

Moved Patterns namespace in its own file.

Support for std::pair

Renamed files to pattern_tools

Fixed support for arbitrary map types.

Renamed namespace, fixed cxx14 dep, fixed stylistic comments.

Added tests for several Convert<T> types.

Add Convert for string types.

Added convert for std::complex types.

Added tensor and point types.

Moved some functions to internal namespaces.

Removed patterns_tools.cc and refactored  add_parameter

Made all stl containers and map available.

Specialization of Convert for Map types.

Added getters to Patterns::Map

Patterns for arithmetic types and container types.

Added Convert struct, with its specialisation for integral types

Added getters for Patterns::List

Minor doc updates to a couple of classes.

Merge pull request #4644 from bangerth/doc-update

Better document the extract_subvector_to() functions.

Explicitly =delete ConstraintMatrix::operator=().

Currently, it was just made 'private' and not implemented.

Add a changelog entry.

Add a test.

Implement a ConstraintMatrix::copy_from() function.

Better document the extract_subvector_to() functions.

This commit deals with the variant of the function that takes iterators.

Better document the extract_subvector_to() functions.

This commit deals with the variant of the function that takes vectors.

Merge pull request #4628 from gassmoeller/allow_zero_dim_output

Allow zero dimensional output in DataOutBase

Add instantiations

Merge pull request #4640 from kronbichler/fix_mg_ghost_periodic

Fix ghost layer with multigrid hierarchy for distributed triangulation

Merge pull request #4636 from bangerth/dofhandler-update

More Dofhandler updates

Instantiate output for zero dim.

Address a couple of review comments.

Standardize on 'template <class DoFHandlerType>'.

Merge pull request #4641 from bangerth/make-var-const

Make a couple of variables 'const'.

Make a couple of variables 'const'.

Add changelog.

Add new test

Fix bug in MG ghost layer computation leading to dead locks.

Merge pull request #4637 from bangerth/fix-serialization

Fix serialization of DoFHandler.

Add class ImmersedSurfaceQuadrature

Add a class representing an quadrature rule over a surface cutting arbitrary
through the unit cell. In order to map a quadrature rule over a surface from
the unit cell to physical space it is not sufficient to know only points and
scalar weights, since information about the normal is also required. In
addition to points and weights this class also stores the normal vectors.

Merge pull request #4634 from bangerth/simplify-code

Simplify a small piece of code.