[PATCH] D72144: Treat C# `using` as a control statement

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 22 08:31:45 PST 2020


krasimir accepted this revision.
krasimir added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/unittests/Format/FormatTestCSharp.cpp:259
+
+  verifyFormat("using (StreamWriter sw = new StreamWriter(filenameB)) {}",
+               Style);
----------------
jbcoe wrote:
> krasimir wrote:
> > Can this appear at the top-level? If not, just keep the other test case to avoid confusion.
> I'm following the example set by by test cases above. I can remove (all) if you prefer and agree it does not test anything new.
I didn't notice the old example. Yes, please remove all of them.


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

https://reviews.llvm.org/D72144





More information about the cfe-commits mailing list