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.stdout.write('[%s/%s] Running: %s' % (index+1, total_file_count, ' 
'.join(invocation) + '\n'))
     subprocess.call(invocation)
----------------
1. Can we have these messages printed on the same line when run in a terminal? 
Progress display is better when it doesn't fill the whole terminal scrollback. 
Changing '\n' to '\r' should be a good start, but we also need to insure 
reasonable interaction with other messages.

2. The whole invocation may be quite a large line. Maybe just print the 
filename? (the rest would be repeated anyway)


https://reviews.llvm.org/D37479



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to