Mesa (master): radeon: Exchange one curious idiom for another in radeonMakeCurrent

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Tue Apr 6 16:44:31 UTC 2021


Module: Mesa
Branch: master
Commit: a6f1d56b7b1cfd955bd79a6e4415d0311b84d007
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a6f1d56b7b1cfd955bd79a6e4415d0311b84d007

Author: Adam Jackson <ajax at redhat.com>
Date:   Mon Mar 22 18:15:48 2021 -0400

radeon: Exchange one curious idiom for another in radeonMakeCurrent

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 <marek.olsak at amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/10045>

---

 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 {



More information about the mesa-commit mailing list