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

Was able to fix this, will post new code with a test for it.


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

Reply via email to