klimek added inline comments.
================
Comment at: clang/tools/clang-format/ClangFormat.cpp:345
if (WarnFormat && !NoWarnFormat) {
+ ArrayRef<std::pair<unsigned, unsigned>> Ranges;
for (const auto &R : Replaces) {
----------------
Looks unused?
================
Comment at: clang/tools/clang-format/ClangFormat.cpp:351
+ SourceLocation LineBegin = Sources.translateFileLineCol(
+ FileEntryPtr.get(), PLoc.getLine() - 1, 0);
+ SourceLocation NextLineBegin =
----------------
This is unexpected: I'd have expected this to be PLoc.getLine() here and
PLoc.getLine() + 1 below. I'm probably missing something?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68969/new/
https://reviews.llvm.org/D68969
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
