Page MenuHomeFreeBSD

EVENTHANDLER(9): drop the list of event handlers
ClosedPublic

Authored by mhorne on Oct 6 2022, 6:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 23 2023, 1:56 AM
Unknown Object (File)
Dec 12 2023, 4:39 PM
Unknown Object (File)
Nov 26 2023, 9:11 AM
Unknown Object (File)
Nov 14 2023, 2:30 PM
Unknown Object (File)
Nov 11 2023, 10:27 AM
Unknown Object (File)
Nov 5 2023, 5:40 PM
Unknown Object (File)
Oct 13 2023, 1:30 PM
Unknown Object (File)
Oct 10 2023, 9:25 AM
Subscribers

Details

Summary

Maintaining a comprehensive list of event handlers in this man page is a
futile endeavor. It is entirely detached from the source code, and
therefore requires that anyone adding/removing an event handler have
prior knowledge of the list. Many do not, so it will naturally become
stale (and is).

This is demonstrated by the fact that there are currently 88 instances
of EVENTHANDLER_DECLARE() in the source tree, but the list contains 66
items.

Many of the descriptions do not offer much detail that could not be
gleaned from the handler name alone. It is a more effective strategy to
document the purpose/details of the event handler in a comment alongside
its declaration.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mhorne requested review of this revision.Oct 6 2022, 6:54 PM

I think this is fine. Without parameter info and some info about what kind of locks are held/may be acquired, this listing isn't useful. Certainly I always just look at the code.

This revision is now accepted and ready to land.Oct 6 2022, 7:47 PM