localspook wrote: > Also regarding splitting into two checks, should I do that or just rename and > leave the requires expression extension for a follow-up PR?
Requires expressions should be a separate PR. I'm personally in favour of adding them as a separate check instead of as options. With separate checks, the weird state of "user enabled the check but disabled all its options" is impossible. > So did we agree to rename to `readability-redundant-parameter-list`? I agree that `parameter-list` is a bit clearer than `parentheses`. But I think what @unterumarmung had in mind was `readability-redundant-lambda-parameter-list`? https://github.com/llvm/llvm-project/pull/190438 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
