i965: Update the comment about viewport hacks.
authorKenneth Graunke <kenneth@whitecape.org>
Fri, 27 Sep 2013 17:13:55 +0000 (10:13 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Sun, 13 Oct 2013 07:10:43 +0000 (00:10 -0700)
It wasn't clear that this was necessary for EGL, or why.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/mesa/drivers/dri/i965/intel_context.c

index 5a73e283c762ce490206d739e86f3ad16a649853..850b18cd65e63145669dc9b37a892bde24db33f0 100644 (file)
@@ -373,7 +373,13 @@ intelInitContext(struct brw_context *brw,
    int bo_reuse_mode;
    struct gl_config visual;
 
-   /* Can't rely on invalidate events, fall back to glViewport hack */
+   /* GLX uses DRI2 invalidate events to handle window resizing.
+    * Unfortunately, EGL does not - libEGL is written in XCB (not Xlib),
+    * which doesn't provide a mechanism for snooping the event queues.
+    *
+    * So EGL still relies on viewport hacks to handle window resizing.
+    * This should go away with DRI3000.
+    */
    if (!driContextPriv->driScreenPriv->dri2.useInvalidate) {
       brw->saved_viewport = functions->Viewport;
       functions->Viewport = intel_viewport;