This pattern is more common and easier to understand.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
Looks ok to me, although, since I know nothing about the interface,
I've taken myself off the review.
sys/nlm/nlm_prot_impl.c | ||
---|---|---|
2404 ↗ | (On Diff #39019) | I think you should actually do the nlm_uninit in MOD_UNLOAD case (modules get MOD_UNLOAD invoked if MOD_LOAD fails). It's fine to have that still fallthrough though. |