This revision was automatically updated to reflect the committed changes.
Closed by commit rG2bd6974aaa66: [clang-format] Wrap lines for C# property
accessors (authored by Jonathan Coe ).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.or
MyDeveloperDay accepted this revision.
MyDeveloperDay added a comment.
LGTM
Comment at: clang/unittests/Format/FormatTestCSharp.cpp:246
- "get;\n"
- "}");
Nit: I feel like this layout should really be an option, (maybe for the
jbcoe marked an inline comment as done.
jbcoe added inline comments.
Comment at: clang/unittests/Format/FormatTestCSharp.cpp:246
- "get;\n"
- "}");
MyDeveloperDay wrote:
> Nit: I feel like this layout should really be an option,
krasimir accepted this revision.
krasimir added a comment.
This is pretty cool!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75006/new/
https://reviews.llvm.org/D75006
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.l
jbcoe updated this revision to Diff 246235.
jbcoe added a comment.
This revision is now accepted and ready to land.
Simplify logic for merging property accessors
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75006/new/
https://reviews.llvm.org/D75006
Files:
clang/lib/Format/Unwrapped