Skip to content

Hotplug #32

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Hotplug #32

wants to merge 3 commits into from

Conversation

nkovacs
Copy link
Contributor

@nkovacs nkovacs commented Mar 18, 2018

See #8

I'm going to squash this when done, but I didn't want to lose the history for now.

nkovacs added 2 commits March 7, 2018 18:37
Remove filtering, it's better to do it in the callback with
simple ifs.
Enumerate in go code to properly support cancellation and
distinguish enumerated devices.
Add support for deregistering from callback.
Fix memory leak.
@nkovacs
Copy link
Contributor Author

nkovacs commented Mar 18, 2018

The documentation is lying. LIBUSB_HOTPLUG_NO_FLAGS doesn't exist in libusb 1.0.17 (LIBUSB_HOTPLUG_ENUMERATE does). They added it later.

LIBUSB_HOTPLUG_NO_FLAGS was added later, it does not exist
in libusb 1.0.16. Its value is zero, so just use the zero value.
@coveralls
Copy link

Coverage Status

Coverage decreased (-4.9%) to 73.319% when pulling 26ac067 on nkovacs:hotplug into d036636 on google:master.

1 similar comment
@coveralls
Copy link

coveralls commented Mar 18, 2018

Coverage Status

Coverage decreased (-4.9%) to 73.319% when pulling 26ac067 on nkovacs:hotplug into d036636 on google:master.

@zagrodzki zagrodzki mentioned this pull request Sep 4, 2018
Elemecca added a commit to Elemecca/gousb that referenced this pull request Oct 19, 2018
@LitoMore
Copy link

Any updates?

Elemecca added a commit to Elemecca/gousb that referenced this pull request Oct 19, 2021
Elemecca added a commit to Elemecca/gousb that referenced this pull request Oct 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants