phosek accepted this revision. phosek added a comment. LGTM
================ Comment at: clang/tools/scan-build-py/lib/libscanbuild/report.py:420 - with open(filename) as handler: + with open(filename, encoding="utf-8") as handler: for line in handler.readlines(): ---------------- The rest of this file uses `'` rather than `"` for strings, can we keep it consistent? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107887/new/ https://reviews.llvm.org/D107887 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits