JonasToth added a comment.

Unicode works both with python3 and python2 (but seemed to work before too).

Otherwise LG



================
Comment at: clang-tidy/tool/run-clang-tidy.py:169
       failed_files.append(name)
-
-    if is_py2:
----------------
The patch does not apply clean to master because of this part. Could you please 
recheck that?


================
Comment at: clang-tidy/tool/run-clang-tidy.py:171
+      sys.stdout.write(' '.join(invocation) + '\n' + output.decode('utf-8') + 
'\n')
       if err:
+        sys.stderr.write(err.decode('utf-8') + '\n')
----------------
please use `len(err)` here instead, python3 does not work.


https://reviews.llvm.org/D51220



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

Reply via email to