Page MenuHomeFreeBSD
Differential D48412

stand/kshim: Replace devclass_equal with calls to strcmp
ClosedPublic
Actions

Authored by jhb on Fri, Jan 10, 3:22 AM.
Tags
None
Referenced Files
F108295825: D48412.diff
Thu, Jan 23, 3:26 PM
Unknown Object (File)
Tue, Jan 14, 6:53 PM
Unknown Object (File)
Sun, Jan 12, 6:02 PM
Unknown Object (File)
Sun, Jan 12, 5:52 PM
Unknown Object (File)
Sun, Jan 12, 3:21 PM
Unknown Object (File)
Sat, Jan 11, 2:37 PM
Unknown Object (File)
Sat, Jan 11, 11:38 AM
Unknown Object (File)
Sat, Jan 11, 11:23 AM
View All 10 Files
Subscribers
dab

Details

Reviewers
imp
markj
emaste
manu
Commits
rG40d7ba087737: stand/kshim: Replace devclass_equal with calls to strcmp

Diff Detail

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

Event Timeline

jhb created this revision.Fri, Jan 10, 3:22 AM
Herald added a reviewer: manu. · View Herald TranscriptFri, Jan 10, 3:22 AM
Herald added a subscriber: dab. · View Herald Transcript
jhb requested review of this revision.Fri, Jan 10, 3:22 AM
Harbormaster completed remote builds in B61636: Diff 149033.Fri, Jan 10, 3:22 AM
jhb added a parent revision: D48411: stand/usb: Quiet warnings so this builds again.Fri, Jan 10, 3:22 AM
emaste accepted this revision.Fri, Jan 10, 3:45 AM

We lose the tiny a == b optimization but I can't imagine that matters.

This revision is now accepted and ready to land.Fri, Jan 10, 3:45 AM
imp accepted this revision.Fri, Jan 10, 4:08 AM
imp added inline comments.
stand/kshim/bsd_kernel.c
888

So hps was a smart fellow. Why did he do it this odd, tortued way?

jhb added inline comments.Fri, Jan 10, 12:02 PM
stand/kshim/bsd_kernel.c
888

Maybe the a == b check? I'm not sure. It is also odd how many places it was used in places that had nothing to do with devclasses.

markj accepted this revision.Fri, Jan 10, 1:56 PM
Closed by commit rG40d7ba087737: stand/kshim: Replace devclass_equal with calls to strcmp (authored by jhb). · Explain WhySat, Jan 11, 4:04 AM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rG40d7ba087737: stand/kshim: Replace devclass_equal with calls to strcmp.

Revision Contents
Changeset List

PathSize
stand/
kshim/
27 lines

Diff 149117

View Options

stand/kshim/bsd_kernel.c

Loading...