curdeius accepted this revision. curdeius added a comment. This revision is now accepted and ready to land.
LGTM. My naming suggestion is not binding. I have no strong opinion on this, but a shorter name would get my 👍 :). So please sync with other reviewers. ================ Comment at: clang/docs/ClangFormatStyleOptions.rst:4007 + * ``bool AfterRequiresKeywordInRequiresClause`` If ``true``, put space between requires keyword in a requires clause and + opening parentheses, if there is one. ---------------- Wouldn't `AfterRequiresInClause`/`AfterRequiresInExpression` be meaningful enough? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113369/new/ https://reviews.llvm.org/D113369 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
