Skip to content

wayland: internal event buffer & wait for xdg configure#255

Merged
tomaka merged 1 commit intorust-windowing:masterfrom
elinorbgr:master
Aug 31, 2017
Merged

wayland: internal event buffer & wait for xdg configure#255
tomaka merged 1 commit intorust-windowing:masterfrom
elinorbgr:master

Conversation

@elinorbgr
Copy link
Copy Markdown
Contributor

This PR does three things:

  • fix Wayland panic with renderer #254 by properly waiting for the configure xdg event from the compositor, this is done by introducing an internal buffer to collect events dispatched internally, while Window::new() may be blocking
  • automatically insert a WindowEvent::Refresh in this internal buffer whenever a window is created, this guarantees the event loop will have something to dispatch when first started, and should solve for good any issue of deadlock of the event loop as previously saw with wayland (without that, the multiwindow example of glutin still deadlocked)
  • update the wayland_window dependency, which provides a better handling of said configure event

@tomaka
Copy link
Copy Markdown
Contributor

tomaka commented Aug 31, 2017

Triage. I guess there's no reason not to merge this.

@tomaka tomaka merged commit 1b22e39 into rust-windowing:master Aug 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Wayland panic with renderer

2 participants