[PATCH] D61850: Removed superfluous and slightly annoying newlines in run-clang-tidy's output.

Sven Panne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 13 03:32:19 PDT 2019


svenpanne created this revision.
svenpanne added a reviewer: alexfh.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

The output of clang-tidy itself already has enough newlines, so the resulting output is more in line with the usual compiler output.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D61850

Files:
  clang-tools-extra/clang-tidy/tool/run-clang-tidy.py


Index: clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
===================================================================
--- clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
+++ clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
@@ -170,10 +170,10 @@
     if proc.returncode != 0:
       failed_files.append(name)
     with lock:
-      sys.stdout.write(' '.join(invocation) + '\n' + output.decode('utf-8') + '\n')
+      sys.stdout.write(' '.join(invocation) + '\n' + output.decode('utf-8'))
       if len(err) > 0:
         sys.stdout.flush()
-        sys.stderr.write(err.decode('utf-8') + '\n')
+        sys.stderr.write(err.decode('utf-8'))
     queue.task_done()
 
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61850.199230.patch
Type: text/x-patch
Size: 683 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190513/bfa3d0e2/attachment.bin>


More information about the cfe-commits mailing list