https://github.com/owenca commented:

Now I just realized that we should add an option for the new behavior (except 
questionable test cases). It seems that 
5a611397a2ffdc9f1988e63a6490b0cd64267d65 only affects the formatting for 
`Cpp11BracedListStyle: true`, so we probably can change that option to an 
`enum`:
- Style1 (same as `true` before)
```c++
SomeFunction({// Comment 1
              "first entry",
              // Comment 2
              "second entry"});
```
- Style2 (space between `{` and the leading comment)
```c++
SomeFunction({ // Comment 1
               "first entry",
               // Comment 2
               "second entry"});
```
or
```c++
SomeFunction({ // Comment 1
    "first entry",
    // Comment 2
    "second entry"});
```
- Style3 (same as `false` before)
```c++
SomeFunction({ // Comment 1
               "first entry",
               // Comment 2
               "second entry" });
```
@mydeveloperday WDYT?

https://github.com/llvm/llvm-project/pull/71672
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to