MyDeveloperDay added inline comments.
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1344
addUnwrappedLine();
FormatTok->Type = TT_FunctionLBrace;
parseBlock(/*MustBeDeclaration=*/false);
jbcoe wrote:
> MyDeveloperDay wrote:
>
jbcoe marked 2 inline comments as done.
jbcoe added inline comments.
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1344
addUnwrappedLine();
FormatTok->Type = TT_FunctionLBrace;
parseBlock(/*MustBeDeclaration=*/false);
MyDeveloperDay added inline comments.
Comment at: clang/unittests/Format/FormatTestCSharp.cpp:249
+ "public string Host { set; get; }");
verifyFormat("[TestMethod(\"start\", HelpText = \"Starts the server "
is this just a personal choice? or
MyDeveloperDay added inline comments.
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1344
addUnwrappedLine();
FormatTok->Type = TT_FunctionLBrace;
parseBlock(/*MustBeDeclaration=*/false);
previously set and get would break
This revision was automatically updated to reflect the committed changes.
Closed by commit rG2f9fc8d9718f: [clang-format] Handle C# property accessors
when parsing lines (authored by Jonathan Coe jb...@google.com).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
krasimir accepted this revision.
krasimir added a comment.
This revision is now accepted and ready to land.
Thank you!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78642/new/
https://reviews.llvm.org/D78642
___
cfe-commits mailing list
jbcoe updated this revision to Diff 259409.
jbcoe added a comment.
Format patch
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78642/new/
https://reviews.llvm.org/D78642
Files:
clang/lib/Format/UnwrappedLineFormatter.cpp
clang/lib/Format/UnwrappedLineParser.cpp
jbcoe created this revision.
jbcoe added reviewers: krasimir, MyDeveloperDay.
jbcoe added a project: clang-format.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Improve C# `{ get; set; } = default;` formatting by handling it in the
UnwrappedLineParser rather than trying
