[PATCH] D87587: [clang-format][PR47290] Add MaxUnwrappedLinesForShortNamespace format option

[Marek Kurdej via Phabricator via cfe-commits]

curdeius added inline comments.


================
Comment at: clang/docs/ClangFormatStyleOptions.rst:2463
 
+**MaxUnwrappedLinesForShortNamespace** (``unsigned``)
+  The maximal number of unwrapped lines that a short namespace spans.
----------------
MyDeveloperDay wrote:
> This is quite a mouthful before this lands do you want to consider shortening it?
> 
> `ShortNamespaceLength` ?
> 
> I'm not sure non clang-formast developers will know what an UnwrappedLine is?
I agree that one needs to wrap its head around to digest this name. I'd vote for what @mydeveloperday proposed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87587/new/

https://reviews.llvm.org/D87587