Skip to content

GPURenderState inconsistencies with the GPU API #12817

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

Open
ritalat opened this issue Apr 13, 2025 · 0 comments
Open

GPURenderState inconsistencies with the GPU API #12817

ritalat opened this issue Apr 13, 2025 · 0 comments

Comments

@ritalat
Copy link
Contributor

ritalat commented Apr 13, 2025

The recently added SDL GPU extension for the 2D Render API has some inconsistencies with GPU API conventions:

  • The GPU API uses Vulkan-ish *CreateInfo structures but the SDL_CreateGPURenderState equivalent is named SDL_GPURenderStateDesc instead
  • Said SDL_GPURenderStateDesc is versioned and must be initialized with SDL_INIT_INTERFACE, while the GPU API *CreateInfo structures have a field for extension properties instead (see SDL_GPUShaderCreateInfo)
  • SDL_SetRenderGPUState should be named SDL_SetGPURenderState(?); all other GPU related renderer functions or structures are named in the style of *GPURender*
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

No branches or pull requests

1 participant