Analysis: (CA1002) Do not expose generic lists.

EditorExtensions/BrowserLink/PixelPushing/PixelPushingMode.cs

@@ -45,7 +45,7 @@ public static bool IsPixelPushingModeEnabled
                 }
             }
         }
-        public static List<Regex> IgnoreList
+        public static IEnumerable<Regex> IgnoreList
         {
             get
             {

EditorExtensions/BrowserLink/UnusedCss/RawRuleUsage.cs

@@ -9,7 +9,7 @@ public class RawRuleUsage : IEquatable<RawRuleUsage>
         [JsonProperty]
         public string Selector { get; set; }
         [JsonProperty]
-        public List<SourceLocation> SourceLocations { get; private set; }
+        public IEnumerable<SourceLocation> SourceLocations { get; private set; }
 
         public RawRuleUsage()
         {

EditorExtensions/BrowserLink/UnusedCss/SessionResult.cs

@@ -18,7 +18,7 @@ public class SessionResult : IUsageDataSource, IResolutionRequiredDataSource
         [JsonProperty]
         public bool Continue { get; set; }
         [JsonProperty]
-        public List<string> Sheets { get; private set; }
+        public IEnumerable<string> Sheets { get; private set; }
 
         public IEnumerable<IStylingRule> AllRules
         {

EditorExtensions/BrowserLink/UnusedCss/UnusedCssExtension.cs

@@ -21,13 +21,13 @@ public class UnusedCssExtension : BrowserLinkExtension
         public static bool IsAnyConnectionAlive { get { return ExtensionByConnection.Count > 0; } }
         public BrowserLinkConnection Connection { get { return _connection; } }
         public bool IsRecording { get; private set; }
-        public static List<string> IgnoreList
+        public static IEnumerable<string> IgnoreList
         {
             get
             {
                 var ignorePatterns = WESettings.GetString(WESettings.Keys.UnusedCss_IgnorePatterns) ?? "";
 
-                return ignorePatterns.Split(new[] { ';' }, StringSplitOptions.RemoveEmptyEntries).Select(x => x.Trim()).ToList();
+                return ignorePatterns.Split(new[] { ';' }, StringSplitOptions.RemoveEmptyEntries).Select(x => x.Trim());
             }
         }
         private static List<string> IgnorePatternList

EditorExtensions/Commands/Code/IntellisenseWriter.cs

@@ -164,12 +164,17 @@ public class IntellisenseObject
         public string FullName { get; set; }
         public bool IsEnum { get; set; }
         public string Summary { get; set; }
-        public List<IntellisenseProperty> Properties { get; private set; }
+        public IList<IntellisenseProperty> Properties { get; private set; }
 
         public IntellisenseObject()
         {
             Properties = new List<IntellisenseProperty>();
         }
+
+        public IntellisenseObject(IList<IntellisenseProperty> properties)
+        {
+            Properties = properties;
+        }
     }
 
     public class IntellisenseProperty

EditorExtensions/Commands/Code/ScriptIntellisenseListener.cs

@@ -163,19 +163,18 @@ private static void ProcessEnum(CodeEnum element, List<IntellisenseObject> list)
 
         private static void ProcessClass(CodeClass cc, List<IntellisenseObject> list)
         {
-            var props = GetProperties(cc.Members, new HashSet<string>()).ToList();
+            var properties = GetProperties(cc.Members, new HashSet<string>()).ToList();
 
-            if (props.Any())
+            if (properties.Any())
             {
-                var intellisenseObject = new IntellisenseObject
+                var intellisenseObject = new IntellisenseObject(properties)
                 {
                     Namespace = GetNamespace(cc),
                     Name = cc.Name,
                     FullName = cc.FullName,
                     Summary = GetSummary(cc),
                 };
 
-                intellisenseObject.Properties.AddRange(props);
                 list.Add(intellisenseObject);
             }
         }