rudolflovrencic wrote: As I was implementing this, I couldn't help to think that it would make it more consistent to have option `EmptyAndAttached` for each `AllowShort*`. This would make it consistent with `AllowShortRecordOnASingleLine` ([docs](https://clang.llvm.org/docs/ClangFormatStyleOptions.html#allowshortrecordonasingleline)) that is also introduced in `v23.0`?
@owenca Are you against converting all these `AllowShort*` bool options to enums and having four options for each (`Never`, `EmptyAndAttached`, `Empty`, `Always`)? https://github.com/llvm/llvm-project/pull/196021 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
