Page MenuHomeFreeBSD

newbus: Remove redundant check for 0
ClosedPublic

Authored by imp on Mar 7 2024, 7:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 18 2024, 7:01 PM
Unknown Object (File)
Dec 2 2024, 1:58 PM
Unknown Object (File)
Nov 6 2024, 12:42 PM
Unknown Object (File)
Nov 6 2024, 10:22 AM
Unknown Object (File)
Oct 20 2024, 9:39 PM
Unknown Object (File)
Oct 20 2024, 9:39 PM
Unknown Object (File)
Oct 20 2024, 9:38 PM
Unknown Object (File)
Oct 20 2024, 9:38 PM
Subscribers

Details

Summary

We already checked to see if the return code is 0 above. This code is
redundant.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 56502
Build 53390: arc lint + arc unit

Event Timeline

imp requested review of this revision.Mar 7 2024, 7:15 PM

actually fix the redundancy

sys/kern/subr_bus.c
1670

I suppose this line could also be dropped.

1739

Maybe we finally need to implement strdup above.

jhb added inline comments.
sys/kern/subr_bus.c
1670

Yes, you can drop pri == 0

1671

out is ok, but you could also use something like exact_match or some other name for the label.

1739

Lots of description strings are constants rather than dynamically allocated. I don't think it would matter if we started always dynamically allocating them though.

This revision is now accepted and ready to land.Mar 13 2024, 8:40 PM
This revision was automatically updated to reflect the committed changes.