jdemeule added a comment.

In D43500#1546089 <https://reviews.llvm.org/D43500#1546089>, @lebedev.ri wrote:

> 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` ?


If I remember by calling 'format::cleanupAroundReplacements' when applying the 
fix in memory.


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

Reply via email to