[Mesa-dev] [PATCH mesa] glx: remove dead code

Adam Jackson ajax at redhat.com
Wed Sep 6 16:17:00 UTC 2017


On Fri, 2017-09-01 at 15:04 +0100, Eric Engestrom wrote:
> These fields were added in 2d94601582 but never used; hasPresent was
> never set, while the other ones were set but never read.

I think this patch is wrong:

> -   dri3_reply = xcb_dri3_query_version_reply(c, dri3_cookie, &error);
> -   if (!dri3_reply) {
> -      free(error);
> -      goto no_extension;
> -   }

You're not just removing the dead stores into the display state, you're
also removing the checks for whether the extensions exist at all. The
DRI3 loader is definitely not going to work if DRI3 and Present aren't,
er, present.

- ajax


More information about the mesa-dev mailing list