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

Incorrect number of written bytes #16

Closed
wants to merge 12 commits into from
Closed

Conversation

jleni
Copy link

@jleni jleni commented Jan 29, 2019

When calling write in Windows, the number of written bytes is off by 1.
This is the result of prepending the HID report ID in Windows.

This different behavior may result in issues in other layers.

@jleni jleni force-pushed the master branch 2 times, most recently from 04f4637 to c9f19e5 Compare January 29, 2019 13:47
@alessio
Copy link

alessio commented May 8, 2019

Hi! Any news on this? @karalabe

@gliptak
Copy link

gliptak commented Mar 15, 2020

#23

@dolmen
Copy link
Contributor

dolmen commented Jan 13, 2022

Commit 302fd40 should be in another pull request, or in fact not present at all as this is probably fixed in a more recent release of hidapi. See #35.

@dolmen
Copy link
Contributor

dolmen commented Jan 14, 2022

This PR is a fix for #36. But the unrelated commit must be removed from the branch.

@jleni
Copy link
Author

jleni commented Jan 14, 2022

I am glad to separate this.. I will do it later today

pennam pushed a commit to pennam/hid that referenced this pull request Jun 29, 2023
rootulp and others added 6 commits July 14, 2023 16:13
Closes #10

Inspired by this PR in upstream: karalabe#18
fix: warning for `sys/poll.h`
It appears the location of the library has moved recently.
OS X enumerate fix: also look for IOKit in /System/Library/Frameworks
@holiman
Copy link
Collaborator

holiman commented Feb 6, 2024

This library is the wrong place to fix it, since it merely pulls in code from https://github.com/libusb/hidapi

@holiman holiman closed this Feb 6, 2024
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.

8 participants