When no InstrumentWorkspace is provided

Giovanni Di Siena authored 5 years ago and Gigg, Martyn Anthony committed 5 years ago

In places other substitutions have been made, e.g
Clang does not yet specialize std::shared_ptr for T[]. Vector
has been used instead. The operator[] methods were incorrectly
marked const but returning a non-const reference - this has been fixed.
Refs #25842

Also updated class_maker.py

This is being done in stages. I've made this change on the Algorithms project
The change consists of removing the manually created DllConfig.h header and replacing
it with one generated during the build by cmake. The MANTID_ALGORITHMS_DLL macro defined
in DllConfig.had was already partially used in the Algorithms project - so some classes
had the attribute DllExport and others already used MANTID_ALGORITHMS_DLL.
I've made it so all classes now use MANTID_ALGORITHMS_DLL.
The DllExport macro was defined in Kernel\System.h so I've removed this include from a
lot of files and replaced it with MantidAlgorithms\DllConfig.h.

As an aside the client projects that use Algorithms such as AlgorithmsTest were working
without this change so it appears that having DllImport present on a class definition
(which is what MANTID_ALGORITHMS_DLL expands to in client executables) is optional

Signed-off-by: Harriet Brown <harriet.brown@stfc.ac.uk>

This commit removes all uses of #ifndef in headers in Algorithms and replaces them with pragma once.

Update copyright headers in all files

Squashed commit of the following:

* First version of a script to manage copyright statements

re #23468

* Neatened up script, added comments

re #23488

* move script to tools directory

re #23488

* small script changes and a couple of manual file changes

re #23488

* Minor chnage to whitespace detection in regex

re #23488

* Add an excluded directory

re #23488

* remove a repeasted copyright statement in a file

re #23488

* Don't comsume the comment end if it is on the same line

re #23488

* fix error in new copright parsing

re #23488

* remove double copyrifght entry

re #23488

* Improve handling of old copyrights at the start of comments

re #23488

* remove empty comments re #23488

* exclude gsoapgenerated directories

re #23488

* Sort out greedy line matching re #23488

* improve empty comment removal

re #23488

* improve false positives

re #23488

* impressive speedup by limiting regex matching length

re #23488

* remove evil invisible non ascii character

Also upadte the copyright at the same time

re #23488

* resolve multiple copyrights in a single file

re #23488

* resolve an issue with new statement detection

re #23488

* another unprintable unicode character

re #23488

* pep updates and cmake the new copyright fit clang format

re #23488

* update already done new format headers

re #23488

* wrong type of bracket

re #23488

* Update class_maker and friends

re #23488

* Update all copyright statements

re #23488

* clang format re #23488

* flake8 warnings re #23488

* Flake8 warnings re #23488

* Exclude .cmake.in and rb.in files

re #23488

* replace missing line re #23488

* exclude .py.in files as they are flasely recognized as C++

re #23488

* another setp.py.in re #23488

* another .py.in correction re #23488

* Hopefully the last of the .py.in files re #23488

* resolve utf-8 encoding of python files and changed ABINS checksum

re #23488

* updates to unit tests that reference line numbers

re #23488

* remaining unit test files and other fixes

re #23488

Update to clang-format-5 and reformat the codebase 

by using the ADS::retrieveWS<Type>() method.

Refs #2840: PeaksWorkspace rewritten to use a vector of Peak objects instead. PeakIntegration should be checked (no test). Added an algorithm that creates a simple empty PeaksWorkspace, for testing and perhaps future use by Andrei.