alecto added a comment. In D132256#3736076 <https://reviews.llvm.org/D132256#3736076>, @HazardyKnusperkeks wrote:
> Do we want to define the behavior of `MaxEmptyLinesToKeep < > DefinitionBlockSpacing`? Then we need tests for that! > Tests with `MaxEmptyLinesToKeep < DefinitionBlockSpacing` and initially more > than `DefinitionBlockSpacing` should definitely be there. Hello! To clarify, do you mean there should be unit tests for this behavior, or do you mean that there should be checks in clang-format itself that will alert the user if MaxEmptyLinesToKeep < DefinitionBlockSpacing? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132256/new/ https://reviews.llvm.org/D132256 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
