Inspection for non-Private Event Handlers and/or implemented Interface Members

[ThunderFrame]

Event Handlers and Implemented Interface members are typically created as Private, but it is possible to declare these members as Friend of Public, which makes those members callable procedures by any consumer of the class.

Rubberduck should warn of such members declared as anything other than explicitly Private.

Implements IFoo
Private WithEvents App As Application

Public Class_Initialize() ' Eeek, it's public, how many times can we initialize?
End Sub

Friend Sub App_AfterCalculate() 'Do we really want to make the AfterCalculate event-handler callable at any time?
End Sub

Public IFoo_Bar() 'Bar is callable against the IFoo interface, but also directly via the class as `IFoo_Bar`
End Sub

[retailcoder]

I want this one to ship for next release.... badly. It's way overdue and plaguing way too much VBA code out there.

[Hosch250]

You'd better hurry up with the inspections refactoring, then.