closes #3052

Author: retailcoder

Rubberduck.Inspections/Concrete/ObjectVariableNotSetInspection.cs

@@ -34,21 +34,22 @@ protected override IEnumerable<IInspectionResult> DoGetInspectionResults()
 
             var objectVariableNotSetReferences = referencesRequiringSetAssignment.Where(FlagIfObjectVariableNotSet);
 
-            return objectVariableNotSetReferences.Select(reference =>
+            return objectVariableNotSetReferences
+                .Select(reference =>
                 new IdentifierReferenceInspectionResult(this,
-                                     string.Format(InspectionsUI.ObjectVariableNotSetInspectionResultFormat, reference.Declaration.IdentifierName),
-                                     State,
-                                     reference));
+                    string.Format(InspectionsUI.ObjectVariableNotSetInspectionResultFormat, reference.Declaration.IdentifierName),
+                    State, reference));
         }
 
         private bool FlagIfObjectVariableNotSet(IdentifierReference reference)
         {
+            var allrefs = reference.Declaration.References;
             var letStmtContext = ParserRuleContextHelper.GetParent<VBAParser.LetStmtContext>(reference.Context);
             var setStmtContext = ParserRuleContextHelper.GetParent<VBAParser.SetStmtContext>(reference.Context);
             var setAssignmentExpression = setStmtContext?.expression()?.GetText();
 
             return reference.IsAssignment && (letStmtContext != null 
-                   || (setAssignmentExpression?.Equals(Tokens.Nothing, StringComparison.InvariantCultureIgnoreCase) ?? false));
+                   || allrefs.All(r => r.IsAssignment && (setAssignmentExpression?.Equals(Tokens.Nothing, StringComparison.InvariantCultureIgnoreCase) ?? false)));
         }
     }
 }

RubberduckTests/Inspections/ObjectVariableNotSetInspectionTests.cs

@@ -27,6 +27,23 @@ End Sub
             AssertInputCodeYieldsExpectedInspectionResultCount(input, expectResultCount);
         }
 
+        [TestMethod]
+        [TestCategory("Inspections")]
+        public void ObjectVariableNotSet_AlsoAssignedToNothing_ReturnsNoResult()
+        {
+            var expectResultCount = 0;
+            var input =
+                @"
+Private Sub DoSomething()
+    Dim target As Object
+    Set target = New Object
+    target.DoSomething
+    Set target = Nothing
+End Sub
+";
+            AssertInputCodeYieldsExpectedInspectionResultCount(input, expectResultCount);
+        }
+
         [TestMethod]
         [TestCategory("Inspections")]
         public void ObjectVariableNotSet_GivenIndexerObjectAccess_ReturnsNoResult()