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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to