Skip to content

Conversation

@antonilol
Copy link
Contributor

@antonilol antonilol commented Oct 27, 2025

  • update sdl2
  • remove platform dependent unused stuff from sdl_bindings.rs
  • update bindgen
  • fix ci? -> later pr
  • check generated sdl_bindings.rs in ci? -> later pr or maybe not

closes #1496
closes #1500

@antonilol antonilol changed the title remove platform dependent unused stuff from sdl_bindings.rs remove platform dependent unused stuff from sdl_bindings.rs, update sdl and dependencies Oct 27, 2025
@antonilol antonilol mentioned this pull request Oct 28, 2025
@jagprog5 jagprog5 requested a review from Cobrand October 28, 2025 16:38
@antonilol
Copy link
Contributor Author

I set it to ready to review earlier, but now it is actually ready.

I also have a suspicion as to why the CI fails, at least only the windows case (not the vcpkg case):
Some X11 event structures are checked by bindgen on windows, from bindings generated on linux. This did not happen before because bindgen used to generate "runtime tests" using #[test] fn test() {}, but now does "compile time tests" that make the compilation fail if the layout of the structures is not expected. I guess a pointer width difference makes it fail?

@antonilol
Copy link
Contributor Author

56a65ed is a small otherwise unrelated change I added (I need Default for keyboard::Mod). It is documented in the changelog as well.

@antonilol antonilol mentioned this pull request Oct 29, 2025
2 tasks
@jagprog5
Copy link
Contributor

looks fine to me. @Cobrand

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.

Segfault when opening (and closing) the controller more than once

2 participants