From c43f57f757840c63f7bf1651616d39e569b1ae3d Mon Sep 17 00:00:00 2001 From: Daniel van Vugt Date: Thu, 8 Feb 2024 17:57:49 +0800 Subject: [PATCH] cogl/onscreen/glx: Fix x/y mixup used to identify the current output I haven't seen this cause a problem but it looks like the worst case is that it would have put the wrong refresh rate value in CoglFrameInfo for multi-monitor systems of differing frequencies. But even that seems unlikely given `_cogl_xlib_renderer_output_for_rectangle` chooses the output with the greatest overlap of the partially-correct rectangle. Part-of: --- cogl/cogl/winsys/cogl-onscreen-glx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cogl/cogl/winsys/cogl-onscreen-glx.c b/cogl/cogl/winsys/cogl-onscreen-glx.c index ce8944074..30f991570 100644 --- a/cogl/cogl/winsys/cogl-onscreen-glx.c +++ b/cogl/cogl/winsys/cogl-onscreen-glx.c @@ -861,7 +861,7 @@ cogl_onscreen_glx_swap_region (CoglOnscreen *onscreen, x_min = CLAMP (x_min, 0, framebuffer_width); x_max = CLAMP (x_max, 0, framebuffer_width); - y_min = CLAMP (y_min, 0, framebuffer_width); + y_min = CLAMP (y_min, 0, framebuffer_height); y_max = CLAMP (y_max, 0, framebuffer_height); output =