jaildesigner-jailgames/coffee_crisis_arcade_edition
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 13 Wiki Activity

fix: Renderizar textura antes de inicializar shaders

Browse Source 
En constructor: SDL_RenderTexture() antes de initShaders()
Esto asegura que la textura tiene contenido válido.

Revertido lazy initialization que no funcionaba.
Vuelta a la solución original que sí funcionaba.
This commit is contained in:
Sergio
2025-10-02 22:00:30 +02:00
parent ba0b0930b0
commit 8d94ed516c
2 changed files with 24 additions and 45 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
source/rendering/opengl/opengl_shader.cpp
View File

@@ -189,34 +189,29 @@ void OpenGLShader::createQuadGeometry() {
}
GLuint OpenGLShader::getTextureID(SDL_Texture* texture) {
if (!texture) {
SDL_LogError(SDL_LOG_CATEGORY_APPLICATION, "texture es nullptr");
return 0;
}
if (!texture) return 1;
SDL_PropertiesID props = SDL_GetTextureProperties(texture);
GLuint texture_id = 0;
// Probar diferentes nombres de properties (Desktop vs ES)
const char* property_names[] = {
SDL_PROP_TEXTURE_OPENGLES2_TEXTURE_NUMBER, // OpenGL ES
SDL_PROP_TEXTURE_OPENGL_TEXTURE_NUMBER, // OpenGL Desktop
"SDL.texture.opengl.texture", // Fallback legacy
"texture.opengl.texture" // Fallback legacy
};
// Intentar obtener ID de textura OpenGL
texture_id = (GLuint)(uintptr_t)SDL_GetPointerProperty(props, "SDL.texture.opengl.texture", nullptr);
for (const char* prop_name : property_names) {
texture_id = (GLuint)SDL_GetNumberProperty(props, prop_name, 0);
if (texture_id != 0) {
SDL_LogInfo(SDL_LOG_CATEGORY_APPLICATION,
"Textura OpenGL obtenida via '%s': ID=%u", prop_name, texture_id);
return texture_id;
}
if (texture_id == 0) {
texture_id = (GLuint)(uintptr_t)SDL_GetPointerProperty(props, "texture.opengl.texture", nullptr);
}
SDL_LogError(SDL_LOG_CATEGORY_APPLICATION,
"No se pudo obtener ID de textura OpenGL con ninguna property");
return 0;
if (texture_id == 0) {
texture_id = (GLuint)SDL_GetNumberProperty(props, "SDL.texture.opengl.texture", 1);
}
if (texture_id == 0) {
SDL_LogWarn(SDL_LOG_CATEGORY_APPLICATION,
"No se pudo obtener ID de textura OpenGL, usando 1 por defecto");
texture_id = 1;
}
return texture_id;
}
bool OpenGLShader::init(SDL_Window* window,
@@ -350,15 +345,6 @@ void OpenGLShader::render() {
// Obtener y bindear textura
GLuint texture_id = getTextureID(back_buffer_);
if (texture_id == 0) {
SDL_LogError(SDL_LOG_CATEGORY_APPLICATION,
"No se pudo obtener texture_id, abortando render");
// Restaurar estados y hacer fallback
SDL_RenderTexture(renderer_, back_buffer_, nullptr, nullptr);
SDL_RenderPresent(renderer_);
return;
}
glEnable(GL_TEXTURE_2D);
glBindTexture(GL_TEXTURE_2D, texture_id);
checkGLError("glBindTexture");

23
source/screen.cpp
View File

@@ -55,8 +55,12 @@ Screen::Screen()
setDebugInfoEnabled(true);
#endif
// Los shaders se inicializarán lazy en el primer render si están activos
// Esto evita problemas con texturas vacías al inicio
// Renderizar una vez la textura vacía para que tenga contenido válido
// antes de inicializar los shaders (evita pantalla negra)
SDL_RenderTexture(renderer_, game_canvas_, nullptr, nullptr);
// Ahora sí inicializar los shaders
initShaders();
}
// Destructor
@@ -96,19 +100,8 @@ void Screen::renderPresent() {
SDL_SetRenderTarget(renderer_, nullptr);
clean();
if (Options::video.shaders) {
// Lazy initialization: inicializar shaders la primera vez que se usan
if (!shader_backend_) {
initShaders();
}
if (shader_backend_) {
shader_backend_->render();
} else {
// Fallback si falla la inicialización
SDL_RenderTexture(renderer_, game_canvas_, nullptr, nullptr);
SDL_RenderPresent(renderer_);
}
if (Options::video.shaders && shader_backend_) {
shader_backend_->render();
} else {
SDL_RenderTexture(renderer_, game_canvas_, nullptr, nullptr);
SDL_RenderPresent(renderer_);