XLibre/xserver
1
1
Fork 0
mirror of https://github.com/X11Libre/xserver.git synced 2026-03-24 01:34:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

glx: vndcmds: save unecessary calls to clear current context for same vendor

Browse Source 
Signed-off-by: sesankm <26676400+sesankm@users.noreply.github.com>
This commit is contained in:
sesankm
2025-09-13 12:08:26 -05:00
committed by Enrico Weigelt
parent dc390bf006
commit 22a51912d1
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
glx/vndcmds.c
View File

@@ -225,9 +225,6 @@ static int CommonMakeCurrent(ClientPtr client,
// The old and new values are all the same, so send a successful reply.
reply.contextTag = oldTag->tag;
} else {
// TODO: For switching contexts in a single vendor, just make one
// makeCurrent call?
// TODO: When changing vendors, would it be better to do the
// MakeCurrent(new) first, then the LoseCurrent(old)?
// If the MakeCurrent(new) fails, then the old context will still be current.
@@ -236,7 +233,7 @@ static int CommonMakeCurrent(ClientPtr client,
// But, if the recovery LoseCurrent(old) fails, then we're really in a bad state.
// Clear the old context first.
if (oldTag != NULL) {
if (oldTag != NULL && oldTag->vendor != newVendor) {
int ret = CommonLoseCurrent(client, oldTag);
if (ret != Success) {
return ret;