goldstein.w.n added a comment. In D137181#3911602 <https://reviews.llvm.org/D137181#3911602>, @owenpan wrote:
> In D137181#3911005 <https://reviews.llvm.org/D137181#3911005>, @sstwcw wrote: > >> Here is one problem: >> >> clang-format -style='{IndentPPDirectives: BeforeHash, PPIndentWidth: 1, >> IndentWidth: 4, IndentCaseLabels: true}' >> >> actual: >> #define X >> \ >> switch (x) { >> \ >> case 0: >> \ >> break; >> \ >> default: >> \ >> break; >> \ >> } >> >> expected: >> #define X >> \ >> switch (x) { >> \ >> case 0: >> \ >> break; >> \ >> default: >> \ >> break; >> \ >> } > > Actually, the expected should be: > > #define X > \ > switch (x) { > \ > case 0: > \ > break; > \ > default: > \ > break; > \ > } Err I'm not sure that's right. I thought we where going for the C-code should start at the column that the 'd' in define is. So it would be: #define X \ switch (x) { \ case 0: \ break; \ default: \ break; \ } Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137181/new/ https://reviews.llvm.org/D137181 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
