[PATCH] D51220: [clang-tidy] run-clang-tidy fails using python 3.7

Andi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 24 09:05:37 PDT 2018


Abpostelnicu added a comment.

In https://reviews.llvm.org/D51220#1212463, @JonasToth wrote:

> I don't know a lot about how to write portable code!
>
> But wouldn't this be out usecase? http://python-future.org/compatible_idioms.html#file-io-with-open
>  Using the `decode` is supposed to work in both pythons


Yes, indeed, this is more elegant solution!


https://reviews.llvm.org/D51220





More information about the cfe-commits mailing list