[PATCH] D76197: clang-format: Use block indentation for braced initializations

2020-04-26 Thread Daan De Meyer via Phabricator via cfe-commits
DaanDeMeyer abandoned this revision. DaanDeMeyer added a comment. Turns out this is already possible by disabling Cpp11BracedListStyle. The proposed change introduced too many changes regardless so I'm abandoning this. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76197/new/ https://re

[PATCH] D76197: clang-format: Use block indentation for braced initializations

2020-03-26 Thread Daan De Meyer via Phabricator via cfe-commits
DaanDeMeyer added a comment. Of course, if there's interest in adding this I'll fix all the tests but I wanted to make sure there was interest in adding this since it changes clang-format's behavior. Can you confirm that this change in behavior is a good thing and might be merged if I fix all t

[PATCH] D76197: clang-format: Use block indentation for braced initializations

2020-03-26 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay requested changes to this revision. MyDeveloperDay added a comment. This revision now requires changes to proceed. > New implementation that breaks fewer tests. `fewer` is not an option, `no` is all thats going to get accepted. Please add tests for you own use case CHANGES SINCE

[PATCH] D76197: clang-format: Use block indentation for braced initializations

2020-03-15 Thread Daan De Meyer via Phabricator via cfe-commits
DaanDeMeyer updated this revision to Diff 250429. DaanDeMeyer added a comment. New implementation that breaks fewer tests. It seems to be expected that block kind is `BK_Unknown` for C struct braced init lists. Is that supposed to be the case? There's still quite some tests failures but most of