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