From a6f1d56b7b1cfd955bd79a6e4415d0311b84d007 Mon Sep 17 00:00:00 2001 From: Adam Jackson Date: Mon, 22 Mar 2021 18:15:48 -0400 Subject: [PATCH] radeon: Exchange one curious idiom for another in radeonMakeCurrent MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit radeonMakeCurrent is the only consumer of _mesa_create_framebuffer(), and all we're doing here is making a dummy for the context to hang on to when there is no current drawable. intelMakeCurrent uses _mesa_get_incomplete_framebuffer() for this instead, which is equivalent, and which matches the usage elsewhere in the tree. Reviewed-by: Marek Olšák Part-of: --- src/mesa/drivers/dri/radeon/radeon_common_context.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/radeon/radeon_common_context.c b/src/mesa/drivers/dri/radeon/radeon_common_context.c index fbe8b8ed5d3..becd30b8ce6 100644 --- a/src/mesa/drivers/dri/radeon/radeon_common_context.c +++ b/src/mesa/drivers/dri/radeon/radeon_common_context.c @@ -605,7 +605,7 @@ GLboolean radeonMakeCurrent(__DRIcontext * driContextPriv, } if(driDrawPriv == NULL && driReadPriv == NULL) { - drfb = _mesa_create_framebuffer(&radeon->glCtx.Visual); + drfb = _mesa_get_incomplete_framebuffer(); readfb = drfb; } else {