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

Future-proof struct encodings #526

Merged
merged 3 commits into from
Dec 3, 2023
Merged

Conversation

madsmtm
Copy link
Owner

@madsmtm madsmtm commented Oct 24, 2023

In rust-windowing/glutin#1640 the encoding of the CGLContextObj object changed between versions of macOS. As such, this is something that we must be prepared to handle in our encoding comparison routines.

@madsmtm madsmtm added bug Something isn't working A-objc2 Affects the `objc2`, `objc2-exception-helper` and/or `objc2-encode` crates labels Oct 24, 2023
@madsmtm madsmtm force-pushed the future-proof-struct-encodings branch from 05e6906 to d49a190 Compare December 3, 2023 06:45
@madsmtm madsmtm marked this pull request as ready for review December 3, 2023 06:46
@madsmtm madsmtm added this to the Usable icrate milestone Dec 3, 2023
@madsmtm madsmtm merged commit 4ec5b64 into master Dec 3, 2023
19 checks passed
@madsmtm madsmtm deleted the future-proof-struct-encodings branch December 3, 2023 06:53
madsmtm added a commit to rust-windowing/glutin that referenced this pull request Apr 18, 2024
kchibisov pushed a commit to rust-windowing/glutin that referenced this pull request Apr 18, 2024
- `icrate` has been deprecated in favour of separate crates per
  framework, in our case `objc2-foundation` and `objc2-app-kit`.
- Moved `MainThreadMarker::run_on_main` to free-standing function
  `run_on_main`.
- Changed how features work, this should result in less code that needs
  to be compiled.

Links: madsmtm/objc2#526
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-objc2 Affects the `objc2`, `objc2-exception-helper` and/or `objc2-encode` crates bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant