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

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 22 12:05:39 PST 2020


MyDeveloperDay requested changes to this revision.
MyDeveloperDay added a comment.
This revision now requires changes to proceed.

In D72144#1834265 <https://reviews.llvm.org/D72144#1834265>, @jbcoe wrote:

> Removed test snippets that did not exercise new code.


Actually i would prefer we didn't remove those tests, because we added them because previously someone called out that it had that effect when at the top level

because clang-format can work on a snippet of code it may be that you don't see the outer scope, I see those tests doing no harm and they assert the behaviour that was previously seen to be different for both in scope and toplevel


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

https://reviews.llvm.org/D72144





More information about the cfe-commits mailing list