duuqnd/cave-story-solaris
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Don't use upscaled framebufer if it's not needed

Browse source 
This matches the SDLTexture backend.

Again, it would be really nice if this code weren't duplicated.
This commit is contained in:
Clownacy 2020-09-15 22:14:19 +01:00
parent 97d5373fa5
commit 08b14fa03f
1 changed files with 26 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
src/Backends/Rendering/OpenGL3.cpp
View file

@ -599,6 +599,12 @@ void RenderBackend_DrawScreen(void)
framebuffer_rect.right = framebuffer_surface->width;
framebuffer_rect.bottom = framebuffer_surface->height;
if (upscaled_framebuffer_surface == NULL)
{
Blit(framebuffer_surface, &framebuffer_rect, &window_surface, &window_rect, false);
}
else
{
RenderBackend_Rect upscaled_framebuffer_rect;
upscaled_framebuffer_rect.left = 0;
upscaled_framebuffer_rect.top = 0;
@ -607,6 +613,7 @@ void RenderBackend_DrawScreen(void)
Blit(framebuffer_surface, &framebuffer_rect, upscaled_framebuffer_surface, &upscaled_framebuffer_rect, false);
Blit(upscaled_framebuffer_surface, &upscaled_framebuffer_rect, &window_surface, &window_rect, false);
}
// Target actual screen, and not our framebuffer
glBindFramebuffer(GL_FRAMEBUFFER, 0);
@ -1032,27 +1039,33 @@ void RenderBackend_HandleWindowResize(size_t width, size_t height)
{
size_t upscale_factor = MAX(1, MIN((width + framebuffer_surface->width / 2) / framebuffer_surface->width, (height + framebuffer_surface->height / 2) / framebuffer_surface->height));
size_t upscaled_framebuffer_width = framebuffer_surface->width * upscale_factor;
size_t upscaled_framebuffer_height = framebuffer_surface->height * upscale_factor;
if (upscaled_framebuffer_surface != NULL)
{
RenderBackend_FreeSurface(upscaled_framebuffer_surface);
upscaled_framebuffer_surface = NULL;
}
upscaled_framebuffer_surface = CreateSurface(framebuffer_surface->width * upscale_factor, framebuffer_surface->height * upscale_factor, true);
if (upscale_factor != 1)
{
upscaled_framebuffer_surface = CreateSurface(upscaled_framebuffer_width, upscaled_framebuffer_height * upscale_factor, true);
if (upscaled_framebuffer_surface == NULL)
Backend_PrintError("Couldn't regenerate upscaled framebuffer");
}
// Create rect that forces 4:3 no matter what size the window is
if (width * upscaled_framebuffer_surface->height >= upscaled_framebuffer_surface->width * height) // Fancy way to do `if (width / height >= upscaled_framebuffer->width / upscaled_framebuffer->height)` without floats
if (width * upscaled_framebuffer_height >= upscaled_framebuffer_width * height) // Fancy way to do `if (width / height >= upscaled_framebuffer->width / upscaled_framebuffer->height)` without floats
{
window_rect.right = (height * upscaled_framebuffer_surface->width) / upscaled_framebuffer_surface->height;
window_rect.right = (height * upscaled_framebuffer_width) / upscaled_framebuffer_height;
window_rect.bottom = height;
}
else
{
window_rect.right = width;
window_rect.bottom = (width * upscaled_framebuffer_surface->height) / upscaled_framebuffer_surface->width;
window_rect.bottom = (width * upscaled_framebuffer_height) / upscaled_framebuffer_width;
}
window_rect.left = (width - window_rect.right) / 2;