close
Skip to content

Fixed typo in JoystickImpl.cpp to prevent crash on OS X#765

Merged
eXpl0it3r merged 1 commit into
masterfrom
bugfix/joystick_osx
Jan 10, 2015
Merged

Fixed typo in JoystickImpl.cpp to prevent crash on OS X#765
eXpl0it3r merged 1 commit into
masterfrom
bugfix/joystick_osx

Conversation

@mantognini

Copy link
Copy Markdown
Member

Supersedes #762.

Read for review/testing.

@executionunit, could you test it?

@executionunit

Copy link
Copy Markdown

Can confirm this works as expected. Thanks.

On 6 Jan 2015, at 23:33, Marco Antognini notifications@github.com wrote:

Supersedes #762 #762.

Read for review/testing.

@executionunit https://github.com/executionunit, could you test it?

You can merge this Pull Request by running

git pull https://github.com/LaurentGomila/SFML bugfix/joystick_osx
Or view, comment on, or merge it at:

#765 #765
Commit Summary

Fixed typo in JoystickImpl.cpp to prevent crash on OS X
File Changes

M src/SFML/Window/OSX/JoystickImpl.cpp https://github.com/LaurentGomila/SFML/pull/765/files#diff-0 (4)
Patch Links:

https://github.com/LaurentGomila/SFML/pull/765.patch https://github.com/LaurentGomila/SFML/pull/765.patch
https://github.com/LaurentGomila/SFML/pull/765.diff https://github.com/LaurentGomila/SFML/pull/765.diff

Reply to this email directly or view it on GitHub #765.

@mantognini

Copy link
Copy Markdown
Member Author

Great!

@eXpl0it3r

Copy link
Copy Markdown
Member

This PR has been added to my merge list, meaning it will be merged soon, unless someone raises any concerns.

@eXpl0it3r eXpl0it3r force-pushed the bugfix/joystick_osx branch from 8ad0af7 to 8a8793e Compare January 10, 2015 22:50
@eXpl0it3r eXpl0it3r merged commit 8a8793e into master Jan 10, 2015
@eXpl0it3r eXpl0it3r deleted the bugfix/joystick_osx branch January 10, 2015 22:51
@mantognini mantognini removed their assignment Apr 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants