[PATCH] D75006: [clang-format] Wrap lines for C# property accessors

2020-02-25 Thread Phabricator via Phabricator via cfe-commits
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

[PATCH] D75006: [clang-format] Wrap lines for C# property accessors

2020-02-25 Thread MyDeveloperDay via Phabricator via cfe-commits
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

[PATCH] D75006: [clang-format] Wrap lines for C# property accessors

2020-02-25 Thread Jonathan B Coe via Phabricator via cfe-commits
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,

[PATCH] D75006: [clang-format] Wrap lines for C# property accessors

2020-02-25 Thread Krasimir Georgiev via Phabricator via cfe-commits
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

[PATCH] D75006: [clang-format] Wrap lines for C# property accessors

2020-02-24 Thread Jonathan B Coe via Phabricator via cfe-commits
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