[PATCH] D49851: [clang-tidy] run-clang-tidy add synchronisation to the output
Jonas Toth via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 24 08:35:29 PDT 2018
JonasToth added a comment.
Please dont work in this revision but create a new one (as this one is
already committed)!
Am 24.08.2018 um 17:34 schrieb Andi via Phabricator:
> Abpostelnicu updated this revision to Diff 162387.
>
> https://reviews.llvm.org/D49851
>
> Files:
>
> clang-tidy/tool/run-clang-tidy.py
>
> Index: clang-tidy/tool/run-clang-tidy.py
> ===================================================================
>
> - clang-tidy/tool/run-clang-tidy.py +++ clang-tidy/tool/run-clang-tidy.py @@ -166,10 +166,18 @@ output, err = proc.communicate() if proc.returncode != 0: failed_files.append(name) + + if is_py2: + output_string = output + err_string = err + else: + output_string = str(output, 'utf-8') + err_string = str(err, 'utf-8') + with lock:
> - sys.stdout.write(' '.join(invocation) + '\n' + output + '\n')
> - if err > 0:
> - sys.stderr.write(err + '\n') + sys.stdout.write(' '.join(invocation) + '\n' + output_string + '\n') + if err: + sys.stderr.write(err_string + '\n') queue.task_done()
https://reviews.llvm.org/D49851
More information about the llvm-commits
mailing list