krasimir accepted this revision. krasimir added inline comments. This revision is now accepted and ready to land.
================ Comment at: clang/unittests/Format/FormatTestCSharp.cpp:609 + + verifyFormat(R"(public float? Value;)", Style); // no space before `?`. +} ---------------- Could you also try/add a test for an array of nullable type (from language ref docs [[ https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/builtin-types/nullable-value-types | here ]]) (also an example of using `new` with a nullable type): ```// An array of a nullable type: int?[] arr = new int?[10];``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75261/new/ https://reviews.llvm.org/D75261 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
