[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