Fixes for spaces around C# object initializers

[jbcoe]

Summary

Fix spaces around typename and [] in C# object initializers.

[MyDeveloperDay]

LGTM, thanks once again for the C# patch!

[krasimir]

clang/lib/Format/TokenAnnotator.cpp

2869

This test feels a bit too rigid: you might additionally want to consider new Type<Param> { and new Type /*comment*/ { and new [] /* comment */ {.
For these you might find the MatchingParen and getPreviousNonComment useful. And example for this is below, at line 2880-2885 in javascript handling.

2870

The TT_Unknown is a bit confusing -- why is it needed? If this is a workaround for something, please add it as a comment. We might need to improve the token detection later to allow for more complicated pattern matching.

[krasimir]

clang/unittests/Format/FormatTestCSharp.cpp

379

Is this needed for the formatting below? I find it surprising if this controls the spacing before braces.

[krasimir]

clang/lib/Format/TokenAnnotator.cpp

2869

Sorry, disregard the patterns with /* comment */ above. I guess the spaces around the comment there are enforced at another place.

[MyDeveloperDay]

clang/lib/Format/TokenAnnotator.cpp

2870

+1 that comment what was Left I think we need to be more specific

[jbcoe]

Minor changes to address review comments.

[krasimir]

Thank you!

clang/lib/Format/TokenAnnotator.cpp

bool TokenAnnotator::spaceRequiredBefore(const AnnotatedLine &Line,
   } else if (Style.isCSharp()) {
     // space between type and variable e.g. Dictionary<string,string> foo;
     if (Left.is(TT_TemplateCloser) && Right.is(TT_StartOfName))
       return true;
     // space between keywords and paren e.g. "using ("
     if (Right.is(tok::l_paren))
       if (Left.is(tok::kw_using))
         return spaceRequiredBeforeParens(Left);
+    // space between ']' and '{'
+    if (Left.is(tok::r_square) && Right.is(tok::l_brace))
+      return true;
+    // space before '{' in "new MyType {"
+    if (Left.is(TT_Unknown) && Right.is(tok::l_brace) && Left.Previous &&
+        Left.Previous->is(tok::kw_new))
+      return true;
   } else if (Style.Language == FormatStyle::LK_JavaScript) {
     if (Left.is(TT_JsFatArrow))
       return true;
     // for await ( ...
     if (Right.is(tok::l_paren) && Left.is(Keywords.kw_await) && Left.Previous &&
         Left.Previous->is(tok::kw_for))
       return true;
     if (Left.is(Keywords.kw_async) && Right.is(tok::l_paren) &&

clang/unittests/Format/FormatTestCSharp.cpp

verifyFormat("foreach(var x in y) {\n"
                "}",
                Style);
   verifyFormat("switch(x) {}", Style);
   verifyFormat("do {\n"
                "} while(x);",
                Style);
 }
 
+TEST_F(FormatTestCSharp, CSharpObjectInitializers) {
+  FormatStyle Style = getGoogleStyle(FormatStyle::LK_CSharp);
+  Style.SpaceBeforeParens = FormatStyle::SBPO_Always;
+
+  verifyFormat("Shape[] shapes = new[] {\n"
+               "    new Circle {\n"
+               "        Radius = 2.7281,\n"
+               "        Colour = Colours.Red,\n"
+               "    },\n"
+               "    new Square {\n"
+               "        Side = 101.1,\n"
+               "        Colour = Colours.Yellow,\n"
+               "    },\n"
+               "};",
+               Style);
+}
+
 } // namespace format
 } // end namespace clang