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

Add wrapper for gbm_device_get_format_modifier_plane_count #33

Closed
wants to merge 1 commit into from

Conversation

ids1024
Copy link
Member

@ids1024 ids1024 commented Feb 5, 2024

I'm not sure what to do about the error type here. gbm_dri_get_format_modifier_plane_count does potentially return -1, but doesn't set errno.

Error handling generally looks like it could use some fixes in both gbm.rs and in Mesa...

I'm not sure what to do about the error type here.
`gbm_dri_get_format_modifier_plane_count` does potentially return -1,
but doesn't set `errno`.

Error handling generally looks like it could use some fixes in both
gbm.rs and in Mesa...
@yshui
Copy link

yshui commented May 26, 2024

I found myself needing this as well. What's left to be done here?

@Drakulix
Copy link
Member

I'm not sure what to do about the error type here. gbm_dri_get_format_modifier_plane_count does potentially return -1, but doesn't set errno.

Maybe just put this into an Option then and parse every value <= 0 as None?
It seems reasonable that some modifiers (e.g. INVALID) don't have a defined plane count.

@Drakulix
Copy link
Member

Closed via #46

@Drakulix Drakulix closed this Nov 21, 2024
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.

3 participants