From 443944c48b30145eae6182679fe1d477f0a7141f Mon Sep 17 00:00:00 2001
From: Jeff Teunissen <deek@d2dc.net>
Date: Sun, 11 Feb 2001 16:05:52 +0000
Subject: [PATCH] context_x11.c: whitespace. dga_check.c: Make sure that the
 version of the VidMode extension at runtime is the same version we compiled
 with, to deal with a structure change that occurred between VidMode extension
 versions 1 and 2. If a version 3 is developed, this will have to be fixed
 again.

---
 source/context_x11.c |  2 +-
 source/dga_check.c   | 36 ++++++++++++++++++++++--------------
 2 files changed, 23 insertions(+), 15 deletions(-)

diff --git a/source/context_x11.c b/source/context_x11.c
index 535e992..dab0342 100644
--- a/source/context_x11.c
+++ b/source/context_x11.c
@@ -279,7 +279,7 @@ x11_set_vidmode (int width, int height)
 
 		XF86VidModeGetAllModeLines (x_disp, x_screen, &nummodes, &vidmodes);
 		XF86VidModeGetViewPort (x_disp, x_screen, &screen_width, &screen_height);
-	
+
 		for (i = 0; i < nummodes; i++) {
 			if ((vidmodes[i]->hdisplay == screen_width) &&
 					(vidmodes[i]->vdisplay == screen_height)) {
diff --git a/source/dga_check.c b/source/dga_check.c
index 613cb14..129b412 100644
--- a/source/dga_check.c
+++ b/source/dga_check.c
@@ -40,6 +40,11 @@
 #endif
 #ifdef HAVE_VIDMODE
 # include <X11/extensions/xf86vmode.h>
+# ifdef X_XF86VidModeSetClientVersion
+#  define VIDMODE_VERSION 2
+# else
+#  define VIDMODE_VERSION 1
+# endif
 #endif
 
 #include "dga_check.h"
@@ -54,7 +59,8 @@ qboolean
 VID_CheckDGA (Display * dpy, int *maj_ver, int *min_ver, int *hasvideo)
 {
 #ifdef HAVE_DGA
-	int         event_base, error_base, dgafeat, dummy;
+	int 	event_base, error_base, dgafeat;
+	int 	dummy, dummy_major, dummy_minor, dummy_video;
 
 	if (!XQueryExtension (dpy, "XFree86-DGA", &dummy, &dummy, &dummy)) {
 		return false;
@@ -65,16 +71,16 @@ VID_CheckDGA (Display * dpy, int *maj_ver, int *min_ver, int *hasvideo)
 	}
 
 	if (!maj_ver)
-		maj_ver = &dummy;
+		maj_ver = &dummy_major;
 	if (!min_ver)
-		min_ver = &dummy;
+		min_ver = &dummy_minor;
 
 	if (!XF86DGAQueryVersion (dpy, maj_ver, min_ver)) {
 		return false;
 	}
 
 	if (!hasvideo)
-		hasvideo = &dummy;
+		hasvideo = &dummy_video;
 
 	if (!XF86DGAQueryDirectVideo (dpy, DefaultScreen (dpy), &dgafeat)) {
 		*hasvideo = 0;
@@ -83,7 +89,7 @@ VID_CheckDGA (Display * dpy, int *maj_ver, int *min_ver, int *hasvideo)
 	}
 
 	if (!(dgafeat & (XF86DGADirectPresent | XF86DGADirectMouse))) {
-			return false;
+		return false;
 	}
 
 	return true;
@@ -101,9 +107,9 @@ VID_CheckDGA (Display * dpy, int *maj_ver, int *min_ver, int *hasvideo)
 qboolean
 VID_CheckVMode (Display * dpy, int *maj_ver, int *min_ver)
 {
-#if defined(HAVE_VIDMODE)
-	int         event_base, error_base;
-	int         dummy;
+#ifdef HAVE_VIDMODE
+	int 	event_base, error_base;
+	int 	dummy, dummy_major, dummy_minor;
 
 	if (!XQueryExtension (dpy, "XFree86-VidModeExtension", &dummy, &dummy, &dummy)) {
 		return false;
@@ -113,14 +119,16 @@ VID_CheckVMode (Display * dpy, int *maj_ver, int *min_ver)
 		return false;
 	}
 
-	if (maj_ver == NULL)
-		maj_ver = &dummy;
-	if (min_ver == NULL)
-		min_ver = &dummy;
+	if (!maj_ver)
+		maj_ver = &dummy_major;
+	if (!min_ver)
+		min_ver = &dummy_minor;
 
-	if (!XF86VidModeQueryVersion (dpy, maj_ver, min_ver)) {
+	if (!XF86VidModeQueryVersion (dpy, maj_ver, min_ver))
+		return false;
+
+	if ((!maj_ver) || (*maj_ver != VIDMODE_VERSION))
 		return false;
-	}
 
 	return true;
 #else