Fixed some bugs, and compilation warnings.

WINGs/array.c

@@ -20,17 +20,17 @@
 
 
 typedef struct W_Array {
-    void **items;		      /* the array data */
-    int itemCount;	              /* # of items in array */
-    int allocSize;	              /* allocated size of array */
-    void (*destructor)(void *item);   /* the destructor to free elements */
+    void **items;		 /* the array data */
+    int itemCount;	         /* # of items in array */
+    int allocSize;	         /* allocated size of array */
+    WMFreeDataProc *destructor;  /* the destructor to free elements */
 } W_Array;
 
 
 WMArray*
 WMCreateArray(int initialSize)
 {
-    WMCreateArrayWithDestructor(initialSize, NULL);
+    return WMCreateArrayWithDestructor(initialSize, NULL);
 }
 
 

WINGs/bagtree.c

@@ -395,7 +395,7 @@ WMCreateTreeBag(void)
 
 
 WMBag*
-WMCreateTreeBagWithDestructor(void (*destructor)(void*))
+WMCreateTreeBagWithDestructor(WMFreeDataProc *destructor)
 {
     WMBag *bag;
 
@@ -752,7 +752,7 @@ WMMapBag(WMBag *self, void (*function)(void*, void*), void *data)
 
 
 static int
-findInTree(W_Bag *tree, W_Node *node, int (*function)(void*,void*), void *cdata)
+findInTree(W_Bag *tree, W_Node *node, WMMatchDataProc *function, void *cdata)
 {
     int index;
 
@@ -772,7 +772,7 @@ findInTree(W_Bag *tree, W_Node *node, int (*function)(void*,void*), void *cdata)
 
 
 int
-WMFindInBag(WMBag *self, int (*match)(void*,void*), void *cdata)
+WMFindInBag(WMBag *self, WMMatchDataProc *match, void *cdata)
 {
     return findInTree(self, self->root, match, cdata);
 }

WINGs/fontmanager.c

@@ -32,11 +32,9 @@ changeFontProp(char *fname, char *newprop, int which)
 WMFont *
 WMGetFontPlain(WMScreen *scrPtr, WMFont *font) 
 {
-	WMFont *nfont=NULL;
-	if(!scrPtr|| !font)
-		return NULL;
-	return font;
-	 
+    if(!scrPtr || !font)
+        return NULL;
+    return font;
 }
 
 WMFont *
@@ -72,10 +70,9 @@ WMGetFontItalic(WMScreen *scrPtr, WMFont *font)
 WMFont *
 WMGetFontOfSize(WMScreen *scrPtr, WMFont *font, int size) 
 {
-	WMFont *nfont=NULL;
-	if(!scrPtr || !font || size<1)
-		return NULL;
-	return font;
+    if(!scrPtr || !font || size<1)
+        return NULL;
+    return font;
 }
 
 

WINGs/wcolorpanel.c

@@ -2158,7 +2158,7 @@ wheelRender(W_ColorPanel *panel)
     unsigned char	*ptr;
     RColor		gray;
     unsigned long	ofs = 0;
-    unsigned char	shift = getShift(sizeof(unsigned char));
+    /*unsigned char	shift = getShift(sizeof(unsigned char));*/
     
     image = RCreateImage(colorWheelSize+4, colorWheelSize+4, True);
     if (!image) {

WINGs/wlist.c

@@ -270,7 +270,6 @@ void
 WMClearList(WMList *lPtr)
 {
     int oldSelected = lPtr->selectedItem;
-    int i;
 
     WMEmptyArray(lPtr->items);
 

WINGs/wruler.c

@@ -223,8 +223,6 @@ static void drawRulerOnPixmap(Ruler * rPtr)
 
 static void paintRuler(Ruler * rPtr)
 {
-    WMScreen *screen = rPtr->view->screen;
-
     if (!rPtr->drawBuffer || !rPtr->view->flags.realized)
         return;
 
@@ -337,7 +335,6 @@ static void rulerDidResize(W_ViewDelegate * self, WMView * view)
 static void handleEvents(XEvent * event, void *data)
 {
     Ruler *rPtr = (Ruler *) data;
-    Display *dpy = event->xany.display;
 
     switch (event->type) {
       case Expose:
@@ -487,6 +484,7 @@ void WMSetRulerMargins(WMRuler * rPtr, WMRulerMargins margins)
 
 }
 
+
 WMRulerMargins *
 WMGetRulerMargins(WMRuler * rPtr)
 {
@@ -505,8 +503,10 @@ WMGetRulerMargins(WMRuler * rPtr)
     /*for */
     margins->tabs = rPtr->margins.tabs;
 
+    return margins;
 }
 
+
 Bool
 WMIsMarginEqualToMargin(WMRulerMargins *aMargin, WMRulerMargins *anotherMargin)
 {
@@ -541,7 +541,7 @@ void WMSetRulerOffset(WMRuler * rPtr, int pixels)
 int WMGetRulerOffset(WMRuler * rPtr)
 {
     if (!rPtr)
-    return;
+        return 0; /* what value should return if no ruler? -1 or 0? */
     return rPtr->offset;
 }
 

WINGs/wtext.c

@@ -1902,16 +1902,15 @@ handleTextKeyPress(Text *tPtr, XEvent *event)
     if (((XKeyEvent *) event)->state & ControlMask)
         control_pressed = True;
     buffer[XLookupString(&event->xkey, buffer, 1, &ksym, NULL)] = 0;
-    
+
     switch(ksym) {
 
         case XK_Right: 
-WMScrollText(tPtr, -14);
+            WMScrollText(tPtr, -14);
         case XK_Left: {
-            TextBlock *tb = tPtr->currentTextBlock;
-            int x = tPtr->cursor.x + tPtr->visible.x;
-            int y = tPtr->visible.y + tPtr->cursor.y + tPtr->cursor.h;
-            int w, pos;
+            //TextBlock *tb = tPtr->currentTextBlock;
+            //int x = tPtr->cursor.x + tPtr->visible.x;
+            //int y = tPtr->visible.y + tPtr->cursor.y + tPtr->cursor.h;
 
 #if 0
             if(!tb)
@@ -1931,7 +1930,7 @@ WMScrollText(tPtr, -14);
                 3 + tPtr->visible.y + tPtr->cursor.y 
                     + tPtr->cursor.h - tPtr->vpos);
             if(x == tPtr->cursor.x + tPtr->visible.x) { 
-printf("same %d %d\n", x, tPtr->cursor.x + tPtr->visible.x);
+                printf("same %d %d\n", x, tPtr->cursor.x + tPtr->visible.x);
                 cursorToTextPosition(tPtr, tPtr->visible.x, 
                 3 + tPtr->visible.y + tPtr->cursor.y + tPtr->cursor.h);
             }