[PATCH] D137181: [clang-format] Don't use 'PPIndentWidth' inside multi-line macros

[Owen Pan via Phabricator via cfe-commits]

owenpan added a comment.

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;                                                                \
      }


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137181/new/

https://reviews.llvm.org/D137181