Runs clang-tidy pass by ref across the codebase to fix-up various
warnings we are seeing in cppcheck

Also updated class_maker.py

- Move the boost optional defaults file to API so it can be accessed outside MantidAlgorithms
- Replace the unused InstrumentOptionDefaults struct with a class that uses the boost optional defaults file to get default values from the algorithm or instrument

Re #23501