[clang-tools-extra] r280844 - Fix a few oversights in the clang-tidy VS plugin.

Zachary Turner via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 7 12:41:19 PDT 2016


Author: zturner
Date: Wed Sep  7 14:41:19 2016
New Revision: 280844

URL: http://llvm.org/viewvc/llvm-project?rev=280844&view=rev
Log:
Fix a few oversights in the clang-tidy VS plugin.

Over-zealous cleanup of using statements removed some that were
actually needed.  Also cleaned up a few warnings.

Modified:
    clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckDatabase.cs
    clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckTree.cs
    clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyProperties.cs
    clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyPropertyGrid.cs

Modified: clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckDatabase.cs
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckDatabase.cs?rev=280844&r1=280843&r2=280844&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckDatabase.cs (original)
+++ clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckDatabase.cs Wed Sep  7 14:41:19 2016
@@ -50,7 +50,7 @@ namespace LLVM.ClangTidy
                 foreach (var Check in Checks_)
                 {
                     if (Names.Contains(Check.Name))
-                        throw new ArgumentException(String.Format("Check {0} exists more than once!", Check.Name));
+                        continue;
                     Names.Add(Check.Name);
                 }
             }

Modified: clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckTree.cs
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckTree.cs?rev=280844&r1=280843&r2=280844&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckTree.cs (original)
+++ clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/CheckTree.cs Wed Sep  7 14:41:19 2016
@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.ComponentModel;
+using System.Linq;
 using System.Text;
 
 namespace LLVM.ClangTidy

Modified: clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyProperties.cs
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyProperties.cs?rev=280844&r1=280843&r2=280844&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyProperties.cs (original)
+++ clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyProperties.cs Wed Sep  7 14:41:19 2016
@@ -40,7 +40,6 @@ namespace LLVM.ClangTidy
         static ClangTidyProperties()
         {
             RootProperties_ = new ClangTidyProperties(null);
-            PropertyDescriptor D;
         }
 
         public static ClangTidyProperties RootProperties

Modified: clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyPropertyGrid.cs
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyPropertyGrid.cs?rev=280844&r1=280843&r2=280844&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyPropertyGrid.cs (original)
+++ clang-tools-extra/trunk/clang-tidy-vs/ClangTidy/ClangTidyPropertyGrid.cs Wed Sep  7 14:41:19 2016
@@ -42,13 +42,6 @@ namespace LLVM.ClangTidy
         /// </summary>
         List<KeyValuePair<string, ClangTidyProperties>> PropertyChain_ = null;
 
-        /// <summary>
-        /// A tree representing all the checks that the extension knows about, used
-        /// when serializing a file to intelligently determine when to use wildcard
-        /// include / exclude rules.
-        /// </summary>
-        CheckTree Checks_;
-
         public ClangTidyPropertyGrid()
         {
             InitializeComponent();




More information about the cfe-commits mailing list