[PATCH] D154484: [clang-format] Add an option to remove redundant parentheses

[MyDeveloperDay via Phabricator via cfe-commits]

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