=?utf-8?q?Tomáš?= Slanina <[email protected]>, =?utf-8?q?Tomáš?= Slanina <[email protected]>, =?utf-8?q?Tomáš?= Slanina <[email protected]>, =?utf-8?q?Tomáš?= Slanina <[email protected]>, =?utf-8?q?Tomáš?= Slanina <[email protected]>, =?utf-8?q?Tomáš?= Slanina <[email protected]>, =?utf-8?q?Tomáš?= Slanina <[email protected]>, =?utf-8?q?Tomáš?= Slanina <[email protected]>, =?utf-8?q?Tomáš?= Slanina <[email protected]>, =?utf-8?q?Tomáš?= Slanina <[email protected]> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/[email protected]>
owenca wrote: > It might be worth notifying me of such a large change before I take the time > to polish a solution you were already going to refuse to merge. How did you know that I was "already going to refuse to merge" when myself didn't because I had *not* reviewed your patch? Anyway, we have a long-standing [high bar](https://clang.llvm.org/docs/ClangFormatStyleOptions.html#adding-additional-style-options) for adding new options although it hasn't always been enforced in recent years. Also, it happened from time to time that patches for new options didn't get merged because reviewers (e.g. #118566) or authors themselves (e.g. #150166) had come up with better alternatives. https://github.com/llvm/llvm-project/pull/151970 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
