[PATCH 05/11] xfree86/dga: Make mieq_install Bool

Ville Syrjala syrjala at sci.fi
Sun Jan 23 15:06:43 PST 2011


mieq_installed is used as a boolean, so why not make it such. Also
it's a static variable, so the the explicit zero initialization can
be removed.

Signed-off-by: Ville Syrjala <syrjala at sci.fi>
---
 hw/xfree86/common/xf86DGA.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/hw/xfree86/common/xf86DGA.c b/hw/xfree86/common/xf86DGA.c
index c4b2d7d..9d0c621 100644
--- a/hw/xfree86/common/xf86DGA.c
+++ b/hw/xfree86/common/xf86DGA.c
@@ -53,7 +53,7 @@
 
 static DevPrivateKeyRec DGAScreenKeyRec;
 #define DGAScreenKeyRegistered dixPrivateKeyRegistered(&DGAScreenKeyRec)
-static int mieq_installed = 0;
+static Bool mieq_installed;
 
 static Bool DGACloseScreen(int i, ScreenPtr pScreen);
 static void DGADestroyColormap(ColormapPtr pmap);
@@ -252,7 +252,7 @@ DGACloseScreen(int i, ScreenPtr pScreen)
 
    if (mieq_installed) {
        mieqSetHandler(ET_DGAEvent, NULL);
-       mieq_installed = 0;
+       mieq_installed = FALSE;
    }
 
    FreeMarkedVisuals(pScreen);
@@ -449,7 +449,7 @@ xf86SetDGAMode(
 
    if (!mieq_installed) {
       mieqSetHandler(ET_DGAEvent, DGAHandleEvent);
-      mieq_installed = 1;
+      mieq_installed = TRUE;
    }
 
    return Success;
@@ -472,7 +472,7 @@ DGASetInputMode(int index, Bool keyboard, Bool mouse)
 
       if (!mieq_installed) {
           mieqSetHandler(ET_DGAEvent, DGAHandleEvent);
-          mieq_installed = 1;
+          mieq_installed = TRUE;
       }
    }
 }
-- 
1.7.3.4



More information about the xorg-devel mailing list