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

Mon Jun 17 07:06:01 PDT 2019

lebedev.ri added a comment.

In D43500#1546077 <https://reviews.llvm.org/D43500#1546077>, @jdemeule wrote:

> In D43500#1244518 <https://reviews.llvm.org/D43500#1244518>, @JonasToth wrote:
>
> > What is the status here? Will you continue to work on this patch @jdemeule (which would be great!) ?
>
>
> Hi, now, this patch contains only tests which are already covered with lit tests.
>  The cleanup is now applied in clang-apply-replacements.

How does that work with `clang-tidy -fix` ?

> 
> 
> In D43500#1491369 <https://reviews.llvm.org/D43500#1491369>, @chgans wrote:
> 
>> Hi, Using llvm-8 (llvm ubuntu repo), clang-replacement has improved, but there still a case when a comma is left beyond: if there was only one member init.
> 
> 
> I do not reproduce at master (9.0.0 trunk), do you have an example in mind?

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D43500/new/

https://reviews.llvm.org/D43500