[PATCH] D78915: [clang-format] Improved parser for C# properties

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay added a comment.

I think if this code is only used by C# which the first if suggests,  we should be OK, just tidy up a bit with the old code and I think this is fine.

Thank you, so great to have other C#'ers doing this ;-)  "Clang format all the things..."



================
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1513
                      Keywords.kw_set)) {
-      if (Tok->isOneOf(Keywords.kw_get, Keywords.kw_set))
+      if (Tok->isOneOf(Keywords.kw_get, Keywords.kw_set)) {
         HasGetOrSet = true;
----------------
 the style in LLVM is to not have braces on single line if statements. (not my style but we need to keep to it)


================
Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1563
+      //  --Line->Level;
+      nextToken();
+      break;
----------------
you need to remove old code


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78915/new/

https://reviews.llvm.org/D78915