Fixed scoping bug with unassigned locals

Author: retailcoder

RetailCoder.VBE/Inspections/VariableNotAssignedInspection.cs

@@ -87,7 +87,7 @@ public IEnumerable<CodeInspectionResultBase> GetInspectionResults(IEnumerable<VB
 
                 // identify unassigned procedure-scoped declarations:
                 unassignedDeclarations.AddRange(
-                    locals.Where(local => assignments.All(a => a.Name != local.Name))
+                    locals.Where(local => assignments.All(a => (a.Scope.MemberName + a.Name) != (local.Scope.MemberName + local.Name)))
                           .Select(identifier => new VariableNotAssignedInspetionResult(Name, Severity, identifier.Context.ambiguousIdentifier(), result.QualifiedName)));
 
                 // identify globals assigned in this module:

RetailCoder.VBE/Inspections/VariableNotAssignedInspetionResult.cs

@@ -20,7 +20,7 @@ public override IDictionary<string, Action<VBE>> GetQuickFixes()
             return
                 new Dictionary<string, Action<VBE>>
                 {
-                    {"Remove unused variable", RemoveVariableDeclaration}
+                    {"Remove unassigned variable (and usages)", RemoveVariableDeclaration}
                 };
         }
 
@@ -46,5 +46,10 @@ private void RemoveVariableDeclaration(VBE vbe)
                 module.InsertLines(selection.StartLine, newCodeLines);
             }
         }
+
+        private void RemoveVariableUsages(VBE vbe)
+        {
+            
+        }
     }
 }