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

Jonathan B Coe via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 28 05:00:06 PST 2020


jbcoe marked an inline comment as done.
jbcoe 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.
+    //
----------------
krasimir wrote:
> 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?
I think any expression is valid.


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