MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land.
A thousand times yes! ================ Comment at: clang/docs/ClangFormatStyleOptions.rst:4354 + + * ``RPS_None`` (in configuration: ``None``) + Do not remove parentheses. ---------------- Should this be Leave, meaning "Don't touch anything?" vs None which could be confused with "No Parentheses" ================ Comment at: clang/docs/ClangFormatStyleOptions.rst:4363 + + * ``RPS_DoubleParentheses`` (in configuration: ``DoubleParentheses``) + Replace double, triple, etc. parentheses with single parentheses. ---------------- Honestly I'm fine with this as it is but should this be? to describe what it will become? e.g. "SingleParentheses" Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154484/new/ https://reviews.llvm.org/D154484 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
