Update to clang-format-5 and reformat the codebase 

Also add debug output regarding time within execute.

This reverts commit cca8a3c2.

Necessary for BinaryOperation. A potential issue is the type of the
output --- we simply use that of the first input, which is maybe not
always correct?

to be run over group workspaces.

1 - If an output workspace property is optional and not set do not try
and create an output workspace.
2 - If an output workspace property is optional and not set do not try
and set a name for the property when running the child algorithms.
3 - Child algorithms can be run without adding a temporary workspace
name. This is the desired behaviour when running over a group
workspace.

# 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 to ensure CrystalFieldFunction is available and up to date.

The intermediate results might not be in ADS so the operating
algorithm's workspace properties should not be set by value.

Re #21064

* Implement it in Algorithm.
* The implementation in AlgorithmProxy now inherits from IAlgorithm.

Re #21064