Skip to content

Commit 25f9dfe

Browse files
committed
Ignore event handlers too
1 parent 4eb0543 commit 25f9dfe

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Rubberduck.CodeAnalysis/Inspections/Concrete/UnderscoreInPublicClassModuleMemberInspection.cs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,12 @@ public UnderscoreInPublicClassModuleMemberInspection(RubberduckParserState state
1515

1616
protected override IEnumerable<IInspectionResult> DoGetInspectionResults()
1717
{
18+
var interfaceMembers = State.DeclarationFinder.FindAllInterfaceImplementingMembers().ToList();
19+
var eventHandlers = State.DeclarationFinder.FindEventHandlers().ToList();
20+
1821
var names = State.DeclarationFinder.UserDeclarations(Parsing.Symbols.DeclarationType.Member)
1922
.Where(w => w.ParentDeclaration.DeclarationType == Parsing.Symbols.DeclarationType.ClassModule)
20-
.Where(w => !State.DeclarationFinder.FindAllInterfaceImplementingMembers().Contains(w))
23+
.Where(w => !interfaceMembers.Contains(w) && !eventHandlers.Contains(w))
2124
.Where(w => w.Accessibility == Parsing.Symbols.Accessibility.Public || w.Accessibility == Parsing.Symbols.Accessibility.Implicit)
2225
.Where(w => w.IdentifierName.Contains('_'))
2326
.ToList();

0 commit comments

Comments
 (0)