Finished removing CommentNode from inspections.

Author: comintern

RetailCoder.VBE/Inspections/Abstract/InspectionResultBase.cs

@@ -21,31 +21,29 @@ protected InspectionResultBase(IInspection inspection, Declaration target)
         /// <summary>
         /// Creates a comment inspection result.
         /// </summary>
-        protected InspectionResultBase(IInspection inspection, CommentNode comment)
-            : this(inspection, comment.QualifiedSelection.QualifiedName, null, comment)
+        protected InspectionResultBase(IInspection inspection, QualifiedModuleName qualifiedName)
+            : this(inspection, qualifiedName, null)
         { }
 
         /// <summary>
         /// Creates an inspection result.
         /// </summary>
-        protected InspectionResultBase(IInspection inspection, QualifiedModuleName qualifiedName, ParserRuleContext context, CommentNode comment = null)
+        protected InspectionResultBase(IInspection inspection, QualifiedModuleName qualifiedName, ParserRuleContext context)
         {
             _inspection = inspection;
             _qualifiedName = qualifiedName;
             _context = context;
-            _comment = comment;
         }
 
         /// <summary>
         /// Creates an inspection result.
         /// </summary>
-        protected InspectionResultBase(IInspection inspection, QualifiedModuleName qualifiedName, ParserRuleContext context, Declaration declaration, CommentNode comment = null)
+        protected InspectionResultBase(IInspection inspection, QualifiedModuleName qualifiedName, ParserRuleContext context, Declaration declaration)
         {
             _inspection = inspection;
             _qualifiedName = qualifiedName;
             _context = context;
             _target = declaration;
-            _comment = comment;
         }
 
         private readonly IInspection _inspection;
@@ -59,9 +57,6 @@ protected InspectionResultBase(IInspection inspection, QualifiedModuleName quali
         private readonly ParserRuleContext _context;
         public ParserRuleContext Context { get { return _context; } }
 
-        private readonly CommentNode _comment;
-        public CommentNode Comment { get { return _comment; } }
-
         private readonly Declaration _target;
         public Declaration Target { get { return _target; } }
 
@@ -72,12 +67,8 @@ public virtual QualifiedSelection QualifiedSelection
         {
             get
             {
-                if (_context == null && _comment == null)
-                {
-                    return _target.QualifiedSelection;
-                }
                 return _context == null
-                    ? _comment.QualifiedSelection
+                    ? _target.QualifiedSelection
                     : new QualifiedSelection(_qualifiedName, _context.GetSelection());
             }
         }

RetailCoder.VBE/Inspections/OptionExplicitInspection.cs

@@ -39,7 +39,7 @@ public override IEnumerable<InspectionResultBase> GetInspectionResults()
                 .Where(declaration => ModuleTypes.Contains(declaration.DeclarationType));
 
             var issues = modules.Where(module => !options.Select(option => option.Scope).Contains(module.Scope))
-                .Select(issue => new OptionExplicitInspectionResult(this, issue.QualifiedName.QualifiedModuleName));
+                .Select(issue => new OptionExplicitInspectionResult(this, issue));
 
             return issues;
         }

RetailCoder.VBE/Inspections/Results/OptionExplicitInspectionResult.cs

@@ -12,8 +12,8 @@ public class OptionExplicitInspectionResult : InspectionResultBase
     {
         private IEnumerable<QuickFixBase> _quickFixes; 
 
-        public OptionExplicitInspectionResult(IInspection inspection, QualifiedModuleName qualifiedName) 
-            : base(inspection, new CommentNode(string.Empty, Tokens.CommentMarker, new QualifiedSelection(qualifiedName, Selection.Home)))
+        public OptionExplicitInspectionResult(IInspection inspection, Declaration target)
+            : base(inspection, target)
         { }
 
         public override IEnumerable<QuickFixBase> QuickFixes