fixed bug in Declaration.AddReference causing duplicate references; added override to pick up unambiguous (grammar-wise) procedure calls.

Author: retailcoder

Rubberduck.Parsing/Symbols/Declaration.cs

@@ -56,7 +56,7 @@ public void AddReference(IdentifierReference reference)
             }
 
             if (reference.Context.Parent != _context 
-                && !_references.Select(r => r.Context).Contains(reference.Context)
+                && !_references.Select(r => r.Context).Contains(reference.Context.Parent)
                 && !_references.Any(r => r.QualifiedModuleName == reference.QualifiedModuleName 
                     && r.Selection.StartLine == reference.Selection.StartLine
                     && r.Selection.EndLine == reference.Selection.EndLine

Rubberduck.Parsing/Symbols/IdentifierReferenceListener.cs

@@ -583,6 +583,11 @@ private Declaration FindProjectScopeDeclaration(string identifierName)
 
         #region IVBAListener overrides
 
+        public override void EnterICS_B_ProcedureCall(VBAParser.ICS_B_ProcedureCallContext context)
+        {
+            Resolve(context.certainIdentifier(), _currentScope);
+        }
+
         public override void EnterICS_B_MemberProcedureCall(VBAParser.ICS_B_MemberProcedureCallContext context)
         {
             if (_alreadyResolved.Contains(context))