[PATCH] D37479: run-clang-tidy: Report progress

2021-01-04 Thread Florian Berchtold via Phabricator via cfe-commits
florian.berchtold added a comment. This looks like a cool feature, why is it blocked? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D37479/new/ https://reviews.llvm.org/D37479 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lis

[PATCH] D37479: run-clang-tidy: Report progress

2017-09-06 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added a comment. I think CMake output is good model for run-clang-tidy. https://reviews.llvm.org/D37479 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D37479: run-clang-tidy: Report progress

2017-09-06 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh requested changes to this revision. alexfh added inline comments. This revision now requires changes to proceed. Comment at: clang-tidy/tool/run-clang-tidy.py:155 args.quiet) -sys.stdout.write(' '.join(invocation) + '\n') +sys.

[PATCH] D37479: run-clang-tidy: Report progress

2017-09-05 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added a comment. It'll be also nice to output progress in terminal when all messages will saved into file. https://reviews.llvm.org/D37479 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/

[PATCH] D37479: run-clang-tidy: Report progress

2017-09-05 Thread Kevin Funk via Phabricator via cfe-commits
kfunk added a comment. In https://reviews.llvm.org/D37479#861580, @Eugene.Zelenko wrote: > I think will be good idea to introduce command line option to control > progress display: turn it off, show in absolute numbers or percents. Honestly, I don't think this configurability is necessary. I'd

[PATCH] D37479: run-clang-tidy: Report progress

2017-09-05 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added a comment. I think will be good idea to introduce command line option to control progress display: turn it off, show in absolute numbers or percents. https://reviews.llvm.org/D37479 ___ cfe-commits mailing list cfe-commits@list