[PATCH] D75194: [clang-format] Do not merge very long C# automatic properties

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 28 05:00:00 PST 2020


krasimir added inline comments.


================
Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:489
+    // If the Limit is broken, split the default value onto a new line and
+    // indent it.
+    //
----------------
Why not always (not just when Limit is broken) merge the whole automatic property declaration into 1 line?
Syntactically, this whole thing is a unit:
`MyVeryLongTypeName Name { get; set } = new MyVeryLongTypeName();`
My C# is rusty, what are the things after the `=` that are syntactically allowed in general?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75194





More information about the cfe-commits mailing list