=?utf-8?q?Tomáš?= Slanina <[email protected]> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/[email protected]>
owenca wrote: > Could you please provide the used config too? Might be necessary to add some > more test cases around this It's the default config. No `.clang-format` file. The inconsistency is that empty records are merged but short, non-empty ones are not, even though we don't have an `AllowShortRecordOnASingleLine` option. I've just glanced at the code WRT merging short/empty records (`tryFitMultipleLinesInOne()`, `tryMergeSimpleBlock()`, etc.) and seen quite a few other issues. Let's leave them out of this patch if possible, so please ignore my question. https://github.com/llvm/llvm-project/pull/154580 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
