Skip to content

Commit 3f7c3cd

Browse files
committed
Correctly check param-not-used for events and library procedures/functions
1 parent f39821b commit 3f7c3cd

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

RetailCoder.VBE/Inspections/ParameterNotUsedInspection.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,9 @@ public override IEnumerable<InspectionResultBase> GetInspectionResults()
3838
var builtInHandlers = declarations.FindBuiltInEventHandlers();
3939

4040
var parameters = declarations.Where(parameter => parameter.DeclarationType == DeclarationType.Parameter
41-
&& !(parameter.Context.Parent.Parent is VBAParser.EventStmtContext)
42-
&& !(parameter.Context.Parent.Parent is VBAParser.DeclareStmtContext));
41+
&& parameter.ParentDeclaration.DeclarationType != DeclarationType.Event
42+
&& parameter.ParentDeclaration.DeclarationType != DeclarationType.LibraryFunction
43+
&& parameter.ParentDeclaration.DeclarationType != DeclarationType.LibraryProcedure);
4344

4445
var unused = parameters.Where(parameter => !parameter.References.Any()).ToList();
4546
var quickFixRefactoring =

0 commit comments

Comments
 (0)