Too bad. Making these two kinds of mails go to the same thread is hardly a trivial thing. And completely switching commit notifications to Phabricator is something not very realistic, I guess (at least, at this point).
Giving up. On Thu, Sep 24, 2015 at 3:24 PM, Manuel Klimek <kli...@google.com> wrote: > The biggest problem is that those comments don't go on the cfe-commmits > thread that gets auto-triggered by commits, and we really want to not add > new threads. > > On Thu, Sep 24, 2015 at 4:28 AM Alexander Kornienko <ale...@google.com> > wrote: > >> alexfh added inline comments. >> >> /clang-tools-extra/trunk/clang-tidy/modernize/LoopConvertCheck.cpp:573 >> test6 >> /clang-tools-extra/trunk/clang-tidy/modernize/LoopConvertCheck.cpp:509 >> test5 >> >> Users: >> klimek (Author) >> >> http://reviews.llvm.org/rL248438 >> >> >> >>
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits