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
Added support for (some) Hat/POV axis #1248
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DONE (cannot "request changes" from my own PR...)
src/SFML/Window/OSX/JoystickImpl.cpp
Outdated
break; | ||
|
||
default: | ||
sf::err() << "Unexpected usage for element of Page Generic Desktop: 0x" << std::hex << IOHIDElementGetUsage(element) << std::dec << std::endl; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be disabled in release builds.
Does this need adjustment for other OS? |
daf9bf9
to
6d6e3f3
Compare
As far as I know, only Mac is concerned by this. |
6d6e3f3
to
79d09a9
Compare
I've rebased this. I believe it can be moved to "Ready". |
79d09a9
to
883ec93
Compare
This adds support for POV (of Hat) axes. It was solely tested with a Dualshock 4 but should gracefully fail (with a report on
sf::err
) when the assumptions behind this implementation don't hold.