[clang-format] Handle constructor invocations after new operator in C# correct

[eoanermine]

Summary

• Handle constructor invocations after new operator in C# correct
• Add test for the case with lambda in constructor arguments after new operator

Fixes https://github.com/llvm/llvm-project/issues/56549

[curdeius]

LGTM.

clang/lib/Format/UnwrappedLineParser.cpp

2864

Nit: comments should be full phrases, ending with full stops

[MyDeveloperDay]

Can we land this for you?

[eoanermine]

In D129926#3705342, @MyDeveloperDay wrote:

Can we land this for you?

Yes

clang/lib/Format/UnwrappedLineParser.cpp

2869–2870

Before the patch, this code mistook lambdas in a constructor call for an initialization list.

[eoanermine]

• Add examples to comments
• Reformat comments right

[curdeius]

@eoanermine, please provide your name and email address for the commit message.

[MyDeveloperDay]

@eoanermine ping... we need your name/email before we can commit.

(@curdeius, @owenpan, @HazardyKnusperkeks ) we need to have a policy for this, that if we don't get the name for a commit we are happy to land, that we'll do it the old way, and just mention them, Its frustrating for us to waste our time doing reviews only to fall at the last hurdle. (@aaron.ballman any thoughts)

[aaron.ballman]

In D129926#3788866, @MyDeveloperDay wrote:

@eoanermine ping... we need your name/email before we can commit.

(@curdeius, @owenpan, @HazardyKnusperkeks ) we need to have a policy for this, that if we don't get the name for a commit we are happy to land, that we'll do it the old way, and just mention them, Its frustrating for us to waste our time doing reviews only to fall at the last hurdle. (@aaron.ballman any thoughts)

Good idea on starting a policy for this. I think that policy should be discussed by the community via an RFC because we should be consistent across the project in how we handle this sort of situation, IMO (at least within the clang part of the repo; it'd be weird for the static analyzer to have a different policy from Clang which is different from clang-format). Personally, I think if there's been no response for a month, we're probably fine to commandeer the patch. That should be sufficient time for folks who have gone on vacation to have come back and responded, hopefully. I think the only situation where we might want a more tight timeframe is when the patch is critical (blocking a release kind of thing), but hopefully that situation is so rare as to not require making a policy for it.

[owenpan]

In D129926#3769542, @curdeius wrote:

@eoanermine, please provide your name and email address for the commit message.

We can use "Danil Sidoruk <[e-mail]>" as I did in d8d331bc9710.

clang/lib/Format/UnwrappedLineParser.cpp

 }
 
 void UnwrappedLineParser::parseNew() {
   assert(FormatTok->is(tok::kw_new) && "'new' expected");
   nextToken();
 
   if (Style.isCSharp()) {
     do {
+      // Handle constructor invocation
+      if (FormatTok->is(tok::l_paren))
+        parseParens();
+
+      // Handle array initialization syntax
       if (FormatTok->is(tok::l_brace))
         parseBracedList();
 
       if (FormatTok->isOneOf(tok::semi, tok::comma))
         return;
 
       nextToken();
     } while (!eof());
   }
 

clang/unittests/Format/FormatTestCSharp.cpp

var x = foo(className, $@"some code:
 
   // Interpolated string with embedded multiline string.
   Code = R"(Console.WriteLine($"{string.Join(@",
     ", values)}");)";
 
   EXPECT_EQ(Code, format(Code, Style));
 }
 
+TEST_F(FormatTestCSharp, CSharpNewOperator) {
+  FormatStyle Style = getLLVMStyle(FormatStyle::LK_CSharp);
+
+  verifyFormat("public void F() {\n"
+               "  var v = new C(() => { var t = 5; });\n"
+               "}",
+               Style);
+  verifyFormat("public void F() {\n"
+               "  var v = new C(() => {\n"
+               "    try {\n"
+               "    } catch {\n"
+               "      var t = 5;\n"
+               "    }\n"
+               "  });\n"
+               "}",
+               Style);
+}
+
 TEST_F(FormatTestCSharp, CSharpLambdas) {
   FormatStyle GoogleStyle = getGoogleStyle(FormatStyle::LK_CSharp);
   FormatStyle MicrosoftStyle = getMicrosoftStyle(FormatStyle::LK_CSharp);
 
   verifyFormat(R"(//
 class MyClass {
   Action<string> greet = name => {
     string greeting = $"Hello {name}!";