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

Can we apply the gamepad issue fix before 3.3.5 is released? #324

Closed
hajimehoshi opened this issue Jul 14, 2021 · 3 comments · Fixed by #325
Closed

Can we apply the gamepad issue fix before 3.3.5 is released? #324

hajimehoshi opened this issue Jul 14, 2021 · 3 comments · Fixed by #325

Comments

@hajimehoshi
Copy link
Member

hajimehoshi commented Jul 14, 2021

Now GLFW 3.3 has an issue that specific gamepads crashes the app immediately, and this will be fixed in 3.3.5 (glfw/glfw#1763). This is a pretty serious problem in my library that uses GLFW (e.g. hajimehoshi/ebiten#1229). As we're not sure when 3.3.5 is released, would it be possible to apply the patch before 3.3.5 is released?

CC @dmitshur

@hajimehoshi hajimehoshi changed the title Can we apply the gamepad issue before 3.3.5 is released? Can we apply the gamepad issue fix before 3.3.5 is released? Jul 14, 2021
@dmitshur
Copy link
Member

Given it is a serious issue, I think "backporting" the GLFW (C library) fix for it in go-glfw would be reasonable to do. It looks like a small patch that just removes those lines that send GLFW_INVALID_VALUE.

In this particular case, we can also consider applying a temporary workaround on the side of go-glfw. That is, to modify glfw.Init to accept GLFW_INVALID_VALUE as well until GLFW 3.3.5 is released and go-glfw updates to using it. Perhaps that's easier, since we'd only need to change Go code.

We should keep issue #292 open until this is fully resolved and workarounds are removed.

@hajimehoshi
Copy link
Member Author

hajimehoshi commented Jul 14, 2021

Thanks.

Now invalidValue is an error that should not be handled in go-gl/glfw. Can we change this to be handleable?

EDIT: NVM, acceptError can take invalidError.

@hajimehoshi
Copy link
Member Author

hajimehoshi/ebiten#1229 (comment)

It looks like we also need to fix PollEvents.

I've already reported this issue at the original GLFW issue (glfw/glfw#1763 (comment))

hajimehoshi added a commit that referenced this issue Jul 14, 2021
This error happens when a joystick is connected while the application
is running.

Closes #324
Updates #292
hajimehoshi added a commit that referenced this issue Jul 15, 2021
…327)

This error happens when a joystick is connected while the application
is running.

Closes #324
Updates #292
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 a pull request may close this issue.

2 participants