Update to clang-format-5 and reformat the codebase 

This will allow users to have logging again

and improve the default user properties file.

re #23102

re #23102

re #23102

re #23102

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.

Merge done to enable documentation be be fixed, because of removal of
alias directive and addition of related algorithms directive.

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.

The logic in the check was switched to be negative `or` rather
than negative `and`. The two people I showed the change to thought
that the phrasing in this PR was clearer.

Refs #21251

Now the set of library names to be excluded is a first class parameter
to the method and can be controlled externally.
Refs #0

Now the set of library names to be excluded is a first class parameter
to the method and can be controlled externally.
Refs #0

The plugin loader can now determine the Qt version and find the appropriate
directory.
Refs #20288

Also redo Timer.cpp and XMLFormatter.h to use c++11 constructs.

This commit removes a second copy constructor, which was causing a
compiler warning. The second constructor had been added to avoid a
problem where the copy constructor is hidden by a templated constructor
when the argument is non-const. The templated constructor has the
correct behaviour, though, so the second copy constructor is not
really necessary.