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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
