fixed FunctionReturnValueNotUsed firing on Sub procedures

Author: retailcoder

RetailCoder.VBE/Inspections/FunctionReturnValueNotUsedInspection.cs

@@ -32,16 +32,14 @@ public override IEnumerable<InspectionResultBase> GetInspectionResults()
             var nonInterfaceFunctions = functions.Except(interfaceMembers.Union(interfaceImplementationMembers));
             var nonInterfaceIssues = GetNonInterfaceIssues(nonInterfaceFunctions);
             return interfaceMemberIssues.Union(nonInterfaceIssues);
-            //// Temporarily disabled until fix for lack of context because of new resolver is found...
-            //return new List<InspectionResultBase>();
         }
 
         private IEnumerable<FunctionReturnValueNotUsedInspectionResult> GetInterfaceMemberIssues(IEnumerable<Declaration> interfaceMembers)
         {
             return from interfaceMember in interfaceMembers
                    let implementationMembers =
                        UserDeclarations.FindInterfaceImplementationMembers(interfaceMember.IdentifierName).ToList()
-                   where
+                   where interfaceMember.DeclarationType == DeclarationType.Function &&
                        !IsReturnValueUsed(interfaceMember) &&
                        implementationMembers.All(member => !IsReturnValueUsed(member))
                    let implementationMemberIssues =
@@ -58,7 +56,7 @@ private IEnumerable<FunctionReturnValueNotUsedInspectionResult> GetInterfaceMemb
 
         private IEnumerable<FunctionReturnValueNotUsedInspectionResult> GetNonInterfaceIssues(IEnumerable<Declaration> nonInterfaceFunctions)
         {
-            var returnValueNotUsedFunctions = nonInterfaceFunctions.Where(function => !IsReturnValueUsed(function));
+            var returnValueNotUsedFunctions = nonInterfaceFunctions.Where(function => function.DeclarationType == DeclarationType.Function && !IsReturnValueUsed(function));
             var nonInterfaceIssues = returnValueNotUsedFunctions
                 .Select(function =>
                         new FunctionReturnValueNotUsedInspectionResult(