HazardyKnusperkeks added a comment. Apart from that I'm okay.
================ Comment at: clang/lib/Format/WhitespaceManager.cpp:1428 + // case, it is allowed if there is a replacement for the empty range + // between 2 tokens and another non-empty range at the start of the second + // token. ---------------- HazardyKnusperkeks wrote: > Can you elaborate where the 2 replacements come from, and why we can't make > sure there is no empty range? This is open (at least for me). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154093/new/ https://reviews.llvm.org/D154093 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
