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

Explicit cast from void* to corresponding type #430

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

Explicit cast from void* to corresponding type #430

wants to merge 2 commits into from

Conversation

iakov
Copy link

@iakov iakov commented Jun 4, 2019

Fix compilation without -fpermissive.

@Youw
Copy link

Youw commented Jun 4, 2019

I suggest making this PR to https://github.com/libusb/hidapi
See #373

xobs and others added 2 commits June 5, 2019 09:38
Modern Windows implementations don't allow opening HID devices in
READ/WRITE mode if they are claimed by the system.  Examples of such
devices include keyboards and mice.

However, these devices can be opened without READ/WRITE permissions, in
order to allow sending and receiving feature reports.

If open_device() fails, retry without requesting READ/WRITE permissions.

This inverts the logic of the parameter to open_device().

It is a refactor of #44 by @pqu.

Signed-off-by: Sean Cross <sean@xobs.io>
Fix compilation without -fpermissive
@iakov
Copy link
Author

iakov commented Jun 5, 2019

Created PR to new location libusb/hidapi: libusb/hidapi#4
This one PR is here just for historical reasons: to emphasize how really outdated this project is.

Megamouse pushed a commit to RPCS3/hidapi that referenced this pull request Jan 6, 2023
- by default find_file/find_library doesn't respect CFLAGS/LDFLAGS, and FindIconv fails to find Iconv;
- by explicitly trying to link against `-liconv` - we're checking if library is available in such way;
- additionally: if Iconv is detected as BUILT_IN, no need to explicitly depend on `Iconv::Iconv`;
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.

3 participants