Page MenuHomeFreeBSD

libusb: make hotplug callback handle an int
Needs ReviewPublic

Authored by bapt on Mar 6 2026, 8:09 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Aug 12, 11:41 PM
Unknown Object (File)
Mon, Aug 10, 11:17 AM
Unknown Object (File)
Sat, Aug 8, 6:27 PM
Unknown Object (File)
Sat, Aug 8, 5:02 AM
Unknown Object (File)
Fri, Aug 7, 9:06 AM
Unknown Object (File)
Thu, Aug 6, 8:45 PM
Unknown Object (File)
Sat, Aug 1, 10:01 PM
Unknown Object (File)
Jul 30 2026, 9:41 AM
Subscribers

Details

Reviewers
emaste
Group Reviewers
USB
Summary

Change libusb_hotplug_callback_handle from a pointer to an int,
matching the sourceforge's libusb API.
Handle backward compatibility.

Diff Detail

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

Event Timeline

bapt requested review of this revision.Mar 6 2026, 8:09 AM

this one in particular I would love to get a review, this is the first time I plan with symbol version compatibility

kib added inline comments.
lib/libusb/libusb10_hotplug.c
531

Why not list the default symbols in Symbol.map?

538

This cannot work, you cannot export two different objects under same name+version. The point of versioning is to provide different versions.

this one in particular I would love to get a review, this is the first time I plan with symbol version compatibility

Hi bapt@, please tell after you finish you work so that I can merge this.
Or would you like to help me merge it after your work?

Update: sorry that I didn't notice that it is a different patch.

Hi, symbol versioning patch is merged. Please rebase and do necessary change.

shlib version has been bumped, we can make the change without symver now

Int change was originally committed in D54211/rGce9ced951a0b; should be OK to recommit now that so version has been bumped

@bapt Hi, My version has been merged by @emaste. Could you please close this?