ilya-biryukov added a comment. One major drawback that I see is the lack of indentation (and other format options) in the added code. Should we have this fix at a higher level that can have formatting (either now or in the future)? E.g. in `clangd` directly?
================ Comment at: lib/Sema/SemaStmt.cpp:1208 + llvm::raw_string_ostream NewCasesOS(NewCases); + // Include \n to separate new cases added if there are many. + // This suppresses printing the (long) insert text. ---------------- Maybe always put each `case` on a new line? From my experience, having multiple cases on the same line is not very common. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63085/new/ https://reviews.llvm.org/D63085 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
