HazardyKnusperkeks added inline comments.

================
Comment at: clang/docs/ClangFormatStyleOptions.rst:2463
 
+**MaxUnwrappedLinesForShortNamespace** (``unsigned``)
+  The maximal number of unwrapped lines that a short namespace spans.
----------------
curdeius wrote:
> 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.
Although I already made some changes inside clang-format and single stepped 
through some formations, I still can not explain what those UnwrappedLines are.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87587

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to