diff --git a/glx/vndcmds.c b/glx/vndcmds.c index 2ee446bf4..39a283ffb 100644 --- a/glx/vndcmds.c +++ b/glx/vndcmds.c @@ -226,6 +226,9 @@ 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. @@ -234,7 +237,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 && oldTag->vendor != newVendor) { + if (oldTag != NULL) { int ret = CommonLoseCurrent(client, oldTag); if (ret != Success) { return ret;