Skip to content

STYLE: Remove space between class and member names in C++ source files#367

Merged
N-Dekker merged 1 commit intodevelopfrom
Remove-space-between-class-and-member-name
Nov 30, 2020
Merged

STYLE: Remove space between class and member names in C++ source files#367
N-Dekker merged 1 commit intodevelopfrom
Remove-space-between-class-and-member-name

Conversation

@N-Dekker
Copy link
Copy Markdown
Member

It appears that clang-format has in some cases introduced a space character between ClassName and ::MemberName. This typically seems to have happened when there originally was a line break between the class name and the member name.

This commit fixes the formatting by removing those spaces, as follows:

find . \( -iname *.cxx \) -exec perl -pi -w -e 's/([A-Z][a-z]+) \:\:([A-Z][a-z])/$1::$2/g;' {} \;

Follows ITK:

It appears that clang-format has in some cases introduced a space character between `ClassName` and `::MemberName`. This typically seems to have happened when there originally was a line break between the class name and the member name.

This commit fixes the formatting by removing those spaces, as follows:

    find . \( -iname *.cxx \) -exec perl -pi -w -e 's/([A-Z][a-z]+) \:\:([A-Z][a-z])/$1::$2/g;' {} \;

Follows ITK:
 - pull request InsightSoftwareConsortium/ITK#2096
 - commit: InsightSoftwareConsortium/ITK@734d6f8
@N-Dekker N-Dekker merged commit a133a51 into develop Nov 30, 2020
@N-Dekker N-Dekker deleted the Remove-space-between-class-and-member-name branch November 30, 2020 11:46
N-Dekker added a commit that referenced this pull request Jul 3, 2021
These spaces must have been introduced accidentally, by clang-format.

Similar to pull request #367 commit a133a51 "STYLE: Remove space between class and member names in C++ source files"
N-Dekker added a commit that referenced this pull request Jul 3, 2021
These spaces must have been introduced accidentally, by clang-format.

Similar to pull request #367 commit a133a51 "STYLE: Remove space between class and member names in C++ source files"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant