r338291 - Remove trailing space

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 30 13:36:23 PDT 2018


On Mon, Jul 30, 2018 at 4:24 PM, Michael Kruse
<llvm-commits at meinersbur.de> wrote:
> I think removing trailing space is a good thing. Some editors remove
> any trailing space when saving a file. This shows up in diffs that I
> then have to undo manually.

I've also run into the same issues on occasion. However, this isn't
about whether we dislike trailing spaces or not; it's about NFC churn
for out of tree users who have to try to keep their local patches
clean against upstream. This kind of churn makes for merge conflicts
and this is a the reason the community has traditionally rejected
these large-scale commits in the past. If we want to change that
policy, it should come after community discussion and agreement.

~Aaron


More information about the cfe-commits mailing list