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

Change Arc<glow::Context> to Rc<glow::Context> #3598

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Conversation

emilk
Copy link
Owner

@emilk emilk commented Nov 21, 2023

This is required for Rust 1.72 (for unknown reasons; see #3595), but also for updating to glow 0.13, where the glow::Context is not longer Sync+Send

This is required for Rust 1.72 (for unknown reasons),
but also for the glow 0.13
@emilk emilk added the egui_glow Relates to running egui_glow on native label Nov 21, 2023
@emilk emilk merged commit 7abf8af into master Nov 21, 2023
35 of 36 checks passed
@emilk emilk deleted the emilk/glow-not-sync branch November 21, 2023 16:13
surban added a commit to surban/egui that referenced this pull request Nov 26, 2023
bagavi pushed a commit to bagavi/three-d that referenced this pull request Nov 29, 2023
bagavi pushed a commit to bagavi/three-d that referenced this pull request Nov 29, 2023
1. Used Rc instead of Arc in Context to make it compatible with egui emilk/egui#3598
bagavi pushed a commit to bagavi/three-d that referenced this pull request Nov 29, 2023
1. Used Rc instead of Arc in Context to make it compatible with egui emilk/egui#3598
bagavi pushed a commit to bagavi/three-d that referenced this pull request Dec 2, 2023
1. Used Rc instead of Arc in Context to make it compatible with egui emilk/egui#3598
marshray pushed a commit to marshray/surban.egui that referenced this pull request Dec 4, 2023
emilk pushed a commit that referenced this pull request Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
egui_glow Relates to running egui_glow on native
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant