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

fix: wayland mmap failed on Sway 1.8.1 #368

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

stacyharper
Copy link
Contributor

@stacyharper stacyharper commented Nov 7, 2023

Sway 1.8.1 does not trigger some event as surface.enter before we are asked to render the first frame. This cause this first frame to be rendered with wrong attributes. (but that is better than crashing)

This is solved with Sway master, precisely with commit:

7d2e4a51063ac90f950cb44f141ab391cbcaff5f

layer-shell: enter output before surface is mapped

This sends fractional-scale-v1 events before the first configure
event. That way clients have all of the metadata they need to render
the first frame.

To ensure we don't fails rendering the first frame, even if the compositor does not provide metadata in a correct order, we initialize the window structure with default values.

Sway 1.8.1 does not trigger some event as surface.enter before we are
asked to render the first frame. This cause this first frame to be
rendered with wrong attributes.

This is solved with Sway master, precisely with commit:

> 7d2e4a51063ac90f950cb44f141ab391cbcaff5f
>
> layer-shell: enter output before surface is mapped
>
> This sends fractional-scale-v1 events before the first configure
> event. That way clients have all of the metadata they need to render
> the first frame.

To ensure we don't fails rendering the first frame, even if the
compositor does not provide metadata in a correct order, we initialize
the window structure with default values.
@Cloudef Cloudef merged commit 62657f8 into Cloudef:master Nov 21, 2023
4 checks passed
@Cloudef
Copy link
Owner

Cloudef commented Nov 21, 2023

Thanks

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.

2 participants