Skip to content
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

C++ wrapper: ignore LIBUSB_ERROR_INTERRUPTED from freenect_process_events() #266

Merged
merged 1 commit into from
Jul 16, 2013

Conversation

jnohlgard
Copy link
Contributor

I kept getting errors about "Cannot process freenect events" when trying to
start cppview although glview worked perfectly. It turns out libusb kept
returning error "interrupted" when starting up, this patch ignores all
such errors from libusb.
I do not know libusb well enough to be able to tell whether this workaround
will cause any problems in special cases.
cppview seems to be working, but crashes when pressing ESC or closing the
window with the error "pure virtual method called". I'm still investigating
that error but I do not think it is related to this patch.

Signed-off-by: Joakim Gebart joakim.gebart@jge.se

…ents()

I kept getting errors about "Cannot process freenect events" when trying to
start cppview although glview worked perfectly. It turns out libusb kept
returning error "interrupted" when starting up, this patch ignores all
such errors from libusb.
I do not know libusb well enough to be able to tell whether this workaround
will cause any problems in special cases.
cppview seems to be working, but crashes when pressing ESC or closing the
window with the error "pure virtual method called". I'm still investigating
that error but I do not think it is related to this patch.

Signed-off-by: Joakim Gebart <joakim.gebart@jge.se>
piedar added a commit that referenced this pull request Jul 16, 2013
C++ wrapper: ignore LIBUSB_ERROR_INTERRUPTED from freenect_process_events()

Reviewed-by: Benn Snyder <benn.snyder@gmail.com>
@piedar piedar merged commit 5b4cb1f into OpenKinect:unstable Jul 16, 2013
@piedar piedar mentioned this pull request Jul 24, 2013
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.

2 participants