[PATCH] D43500: [clang-tidy]: modernize-use-default-member-init: Remove trailing comma and colon.

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 22 05:33:37 PST 2018


alexfh added a reviewer: ioeric.
alexfh added a comment.

In https://reviews.llvm.org/D43500#1013497, @aaron.ballman wrote:

> > It seems, when they are apply directly from clang-tidy, the RefactoringTool engine is smart enough to remove trailing tokens. However, when fixes are exported, clang-apply-replacements cannot do the same trick and will lead trailing commas and colon
>
> Is there a way to make clang-apply-replacements smarter rather than forcing every check to jump through hoops? I'm worried that if we have to fix individual checks we'll just run into the same bug later.


Eric was going to look at this, IIRC.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D43500





More information about the cfe-commits mailing list