[PATCH] D51220: [clang-tidy] run-clang-tidy fails using python 3.7
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 24 09:06:34 PDT 2018
JonasToth added a comment.
Could you please verify it is actually working?
Am 24.08.2018 um 18:05 schrieb Andi via Phabricator:
> 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
https://reviews.llvm.org/D51220
More information about the cfe-commits
mailing list