Section 'Class Layout' of coding style guide page says:
#ifndef mitkClassName_h
#define mitkClassName_h
But section 'Include Guards' says:
Naming convention for #include guards is: ClassName_h
Section 'Class Layout' of coding style guide page says:
#ifndef mitkClassName_h
#define mitkClassName_h
But section 'Include Guards' says:
Naming convention for #include guards is: ClassName_h
Restricted Differential Revision |
Thanks for pointing that out. The style guide will need to be adapted anyway once we have decided on a .clang-format convention.
User goch has pushed new remote branch:
bug-19210-update-style-guide-and-code-formatting
Apply clang-format:
for f in $(find . -name '*.cpp' -or -name '*.h' -or -name '*.txx' -or -name '*.hxx' -or -name '*.tpp' -or -name '*.cxx');do "/d/Program Files (x86)/LLVM/bin/clang-format.exe" -style=file -i $f; done
Delete trailing white spaces afterwards:
for f in $(find . -name '*.cpp' -or -name '*.h' -or -name '*.txx' -or -name '*.hxx' -or -name '*.tpp' -or -name '*.cxx');do sed -i -e's/[ \t]*$//g' $f; done
Suggested merge strategy:
git merge -X ignore-space-change
IpFunc... utilities were not converted
Needs core flag.
Changes were only documentation and reformatting. It was tested to compile on windows. I will compile it on Linux as well before merging.