[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