COMP: Address clang-tidy warnings in generated code

[hjmjohnson]

Modernize the generateCLP main code, and generated code.

Updating the generated code allows programs using
SEM to build cleanly with clang-tidy.

[hjmjohnson]

@jcfr Do you have concerns with the change proposed? These changes clear at least 6 warnings for every executable that uses SEM PARSE_ARGS at the begining of the main function.

[jcfr]

Thanks for the reminder and for working on this 🙏 💯

Since the minimum version of C++ standard now supported is 11, the proposed changes make sense.

SlicerExecutionModel/CMakeLists.txt

Line 19 in f19d6e8

set(VALID_CXX_STANDARDS "11" "14" "17")

[jcfr]

Let's address #136 before merging

[hjmjohnson]

@jcfr Can this now be merged? It has been open without substantive changes or concerns since the last august.

[hjmjohnson]

@jcfr PING.

[hjmjohnson]

@jcfr #136 was holding this up, but is now closed. Can we please merge this PR now?

[hjmjohnson]

@jcfr @pieper Any chance that this can be rejected or accepted? It has been passing for nearly a year.

[jcfr]

Thanks for the ping 🙏 Will follow up on this 🚀

[hjmjohnson]

ping.. silent little ping.. barely audible.

[jamesobutler]

@hjmjohnson Are there any additional changes you would like to make here considering Slicer defined the minimum to C++17 since you last pushed changes here. Slicer/Slicer@dedb6c0

[hjmjohnson]

@jamesobutler Perhaps, but I don't want to hold this PR up any longer. It has been 18 months waiting for this step to be approved. I don't want to put any more effort into making this better until I know that previous efforts are considered. I don't know if this PR is rejected.

[jcfr]

Thanks for the reminder 🙏 and patience.

Will integrate today 🚀