]> source.dussan.org Git - tigervnc.git/commitdiff
Eliminated several GCC warnings:
authorPeter Åstrand <astrand@cendio.se>
Mon, 26 Jan 2009 09:10:57 +0000 (09:10 +0000)
committerPeter Åstrand <astrand@cendio.se>
Mon, 26 Jan 2009 09:10:57 +0000 (09:10 +0000)
* Initializing "vis" variable.

* Using dixLookupResourceByType instead of deprecated
dixLookupResource.

* Using unsigned integers when dealing with unsigned numbers.

git-svn-id: svn://svn.code.sf.net/p/tigervnc/code/trunk@3573 3789f03b-4d11-0410-bbf8-ca57d06f2519

unix/xserver/hw/vnc/XserverDesktop.cc

index 203ad5ad93d89c8d6bcd5805a975301e21e52c5f..da4935acf3bfa60ba357a8f22be49113cd92bc16 100644 (file)
@@ -180,7 +180,7 @@ XserverDesktop::XserverDesktop(ScreenPtr pScreen_,
   }
   format.bigEndian = (screenInfo.imageByteOrder == MSBFirst);
 
-  VisualPtr vis;
+  VisualPtr vis = NULL;
   for (i = 0; i < pScreen->numVisuals; i++) {
     if (pScreen->visuals[i].vid == pScreen->rootVisual) {
       vis = &pScreen->visuals[i];
@@ -263,7 +263,7 @@ void XserverDesktop::serverReset(ScreenPtr pScreen_)
   int i;
   pointer retval;
 
-  i = dixLookupResource(&retval, pScreen->defColormap, RT_COLORMAP, NullClient,
+  i = dixLookupResourceByType(&retval, pScreen->defColormap, RT_COLORMAP, NullClient,
                        DixReadAccess);
 
   /* Handle suspicious conditions */
@@ -352,8 +352,8 @@ void XserverDesktop::setColourMapEntries(ColormapPtr pColormap, int ndef,
 {
   if (cmap != pColormap || ndef <= 0) return;
 
-  int first = pdef[0].pixel;
-  int n = 1;
+  unsigned int first = pdef[0].pixel;
+  unsigned int n = 1;
 
   for (int i = 1; i < ndef; i++) {
     if (first + n == pdef[i].pixel) {
@@ -984,7 +984,8 @@ void XserverDesktop::keyEvent(rdr::U32 keysym, bool down)
 {
   KeyClassPtr keyc = vncKeyboardDevice->key;
   KeySymsPtr keymap = &keyc->curKeySyms;
-  int i, j, k, n;
+  unsigned int i, n;
+  int j, k;
 
   if (keysym == XK_Caps_Lock) {
     vlog.debug("Ignoring caps lock");