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

Use new primary scanout output if old output is disabled #990

Merged
merged 2 commits into from
Nov 13, 2024

Conversation

ids1024
Copy link
Member

@ids1024 ids1024 commented Nov 13, 2024

Output in Smithay doesn't track if the output still exists, other than based on whether or not it has strong references. Which doesn't seem to be working correctly.

There may be leaked strong references to Outputs somewhere, and maybe Smithay should track if an output is still valid, generally when it is exposed as a Wayland global
(Smithay/smithay#1584). But a check like this works for now.

Addresses #985.

`add_heads` will set it to `true` again, if used on the same `Output`.
It doesn't like like cosmic-comp's usage of this does that currently
though.
`Output` in Smithay doesn't track if the output still exists, other than
based on whether or not it has strong references. Which doesn't seem to
be working correctly.

There may be leaked strong references to `Output`s somewhere, and maybe
Smithay should track if an output is still valid, generally when it is
exposed as a Wayland global
(Smithay/smithay#1584). But a check like this
works for now.

Addresses #985.
@Drakulix Drakulix merged commit 4db2e3e into master Nov 13, 2024
7 checks passed
@ids1024 ids1024 deleted the primary-scanout-output branch November 14, 2024 00:06
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