Skip to content

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