Fix clang missing override warnings

This MR fixes a lot of missing override clang warning.

assigned to @jmichel

Great idea!! Could we also change all ITK_OVERRIDE with override in this merge request?

Could we also change all ITK_OVERRIDE with override in this merge request?

Well, I tried to mimic the current state : use ITK_OVERRIDE in OTB, except in ossimPlugins.

But it would be fairly easy to sed -e s/ITK_OVERRIDE/override/g ;)

I am okay with the replacement of ITK_OVERRIDE since we enforce C++14 anyway.

Should we do this in this merge request?

added Doing label

added 1 commit

• 756ee5dc - STY: Change ITK_OVERRIDE for override in whole code (find Modules -type f -exec…

Compare with previous version

This MR is ready for merge, I am just missing one vote (@aregimbeau, @lnicola, @ytanguy, @remicress ?) Dashboard is ok appart from one test on one machine which seems to have outdated baseline issues.

I can't really review it, but it doesn't seem like a bad idea.

-   // No need to ITK_OVERRIDE the input kwl, just setup the
+   // No need to override the input kwl, just setup the

Nice.

@jmichel In the context of !21 (merged), I don't think I should be able to vote for a MR (it says "Core developers ('Master' members in Gitlab ; it includes PSC members)", and I am neither), but I'll give it a thumbs up.

added 77 commits

• 756ee5dc...f4aaebfa - 76 commits from branch develop
• 82005eec - Merge branch 'develop' into fix-clang-warnings

Compare with previous version

mentioned in commit f781a7de

merged

removed Doing label

added bug label