<div dir="ltr">The unnecessary usings are added by Visual Studio, so I would rather leave them.  For example, they often provide extension methods to containers so that containers can be used with functional paradigms, and you expect to see those methods in the Intellisense window when you type "Foo.", but without the using statement they would not appear.  In general they don't affect performance or compilation the same way that #includes do in C++, so for the ones that are added automatically by Visual Studio I would rather leave them alone.  I've cleaned up all the unnecessary ones that were not automatically added though.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Sep 7, 2016 at 4:27 AM Alexander Kornienko <<a href="mailto:alexfh@google.com">alexfh@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">alexfh added inline comments.<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: clang-tidy-vs/ClangTidy/ClangTidyPackage.cs:53<br class="gmail_msg">
@@ +52,3 @@<br class="gmail_msg">
+        private void MenuItemCallback(object sender, EventArgs args)<br class="gmail_msg">
+        {<br class="gmail_msg">
+        }<br class="gmail_msg">
----------------<br class="gmail_msg">
Add a FIXME to actually implement this.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D23848" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D23848</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>