[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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits