[PATCH] D49851: [clang-tidy] run-clang-tidy add synchronisation to the output

[Jonas Toth via Phabricator via cfe-commits]

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