This pattern is more common and easier to understand.
Details
Details
Diff Detail
Diff Detail
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 14879 Build 14991: arc lint + arc unit
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 | ||
---|---|---|
2403 | 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. |