[PATCH] D50055: Update the coding standard about NFC changes and whitespace

Michael Kruse via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 4 19:48:49 PDT 2018

Meinersbur added inline comments.

Comment at: docs/CodingStandards.rst:514-516
+line of code. Some common editors will automatically remove trailing whitespace
+when saving a file which causes unrelated changes to appear in diffs and
Just to note that this policy will prohibit the use of remove trailing-whitespace feature in editors since it makes it impossible to edit the file without also 'editing' any unrelated whitespace. At the same time, since not being able to use the feature, I risk committing trailing whitespace myself (that is, some additional steps are necessary: I use 'git clang-format' which should remove traling whitespace only on edited lines and 'git diff' shows trailing whitespace in red; these are still additional steps that are easy to miss)


More information about the cfe-commits mailing list