Close #1531. Remove some dead bug comments. (#1535)

Author: Hosch250

RetailCoder.VBE/Inspections/ObsoleteTypeHintInspection.cs

@@ -22,12 +22,18 @@ public override IEnumerable<InspectionResultBase> GetInspectionResults()
 
             var declarations = from item in results
                 where item.HasTypeHint()
-                // bug: this inspection result only has one value.  Why are we passing two in?
-                               select new ObsoleteTypeHintInspectionResult(this, string.Format(InspectionsUI.ObsoleteTypeHintInspectionResultFormat, InspectionsUI.Inspections_Declaration, item.DeclarationType.ToString().ToLower(), item.IdentifierName), new QualifiedContext(item.QualifiedName, item.Context), item);
-            // todo: localize this InspectionResultFormat properly
+                select new ObsoleteTypeHintInspectionResult(this,
+                        string.Format(InspectionsUI.ObsoleteTypeHintInspectionResultFormat,
+                            InspectionsUI.Inspections_Declaration, item.DeclarationType.ToString().ToLower(),
+                            item.IdentifierName), new QualifiedContext(item.QualifiedName, item.Context), item);
+            
             var references = from item in results.SelectMany(d => d.References)
                 where item.HasTypeHint()
-                             select new ObsoleteTypeHintInspectionResult(this, string.Format(InspectionsUI.ObsoleteTypeHintInspectionResultFormat, InspectionsUI.Inspections_Usage, item.Declaration.DeclarationType.ToString().ToLower(), item.IdentifierName), new QualifiedContext(item.QualifiedModuleName, item.Context), item.Declaration);
+                select new ObsoleteTypeHintInspectionResult(this,
+                        string.Format(InspectionsUI.ObsoleteTypeHintInspectionResultFormat,
+                            InspectionsUI.Inspections_Usage, item.Declaration.DeclarationType.ToString().ToLower(),
+                            item.IdentifierName), new QualifiedContext(item.QualifiedModuleName, item.Context),
+                        item.Declaration);
 
             return declarations.Union(references);
         }

RetailCoder.VBE/Inspections/UntypedFunctionUsageInspectionResult.cs

@@ -32,18 +32,20 @@ public override string Description
     public class UntypedFunctionUsageQuickFix : CodeInspectionQuickFix
     {
         public UntypedFunctionUsageQuickFix(ParserRuleContext context, QualifiedSelection selection) 
-            : base(context, selection, string.Format(InspectionsUI.QuickFixUseTypedFunction_, context.GetText(), context.GetText() + "$"))
+            : base(context, selection,
+                  string.Format(InspectionsUI.QuickFixUseTypedFunction_,
+                      context.GetText(), context.GetText().Insert(context.GetText().IndexOf('('), "$")))
         {
         }
 
         public override void Fix()
         {
             var originalInstruction = Context.GetText();
-            var newInstruction = originalInstruction + "$";
+            var newInstruction = originalInstruction.Insert(originalInstruction.IndexOf('('), "$");
             var selection = Selection.Selection;
 
             var module = Selection.QualifiedName.Component.CodeModule;
-            var lines = module.get_Lines(selection.StartLine, selection.LineCount);
+            var lines = module.Lines[selection.StartLine, selection.LineCount];
 
             var result = lines.Replace(originalInstruction, newInstruction);
             module.ReplaceLine(selection.StartLine, result);