fix some warnings from -Wall

diff --git a/openbox/action.c b/openbox/action.c
index 05a9e640ae1eb682871165c2d5ae985d4d9740ad..c0195dd820bda25358cbdfa733a666e87b70b6ca 100644 (file)
--- a/openbox/action.c
+++ b/openbox/action.c
@@ -29,6 +29,7 @@
 #include "grab.h"
 #include "keyboard.h"
 #include "event.h"
+#include "dock.h"
 #include "config.h"
 #include "mainloop.h"
 

diff --git a/openbox/menuframe.c b/openbox/menuframe.c
index 120f4c385c89e9a75fa2d9e4d16c10e83181c5a2..62a68ae9fbdf43fdd99cdb5644ac2677e80d05c3 100644 (file)
--- a/openbox/menuframe.c
+++ b/openbox/menuframe.c
@@ -23,6 +23,7 @@
 #include "screen.h"
 #include "grab.h"
 #include "openbox.h"
+#include "mainloop.h"
 #include "config.h"
 #include "render/theme.h"
 
@@ -189,7 +190,6 @@ void menu_frame_move(ObMenuFrame *self, gint x, gint y)
 void menu_frame_move_on_screen(ObMenuFrame *self)
 {
     Rect *a = NULL;
-    guint i;
     gint dx = 0, dy = 0;
     gint pos, half;
 

diff --git a/openbox/stacking.c b/openbox/stacking.c
index 714d2a1c1f491a8c479b5acfd0a73c93e53dda19..bedc10c817e2973f144ca54288003302d4b2dd99 100644 (file)
--- a/openbox/stacking.c
+++ b/openbox/stacking.c
@@ -56,11 +56,12 @@ void stacking_set_list()
 
 static void do_restack(GList *wins, GList *before)
 {
-    GList *it, *next;
+    GList *it;
     Window *win;
     gint i;
 
 #ifdef DEBUG
+    GList *next;
     /* pls only restack stuff in the same layer at a time */
     for (it = wins; it; it = next) {
         next = g_list_next(it);

diff --git a/render/font.c b/render/font.c
index 4d9e0adf984d6acdcf9099afbbe4d33d6f60fd3d..79a5b40d671e113b4e9e40ed631a9965ad05ac37 100644 (file)
--- a/render/font.c
+++ b/render/font.c
@@ -291,8 +291,9 @@ void RrFontDraw(XftDraw *d, RrTextureText *t, RrRect *area)
     gint x,y,w,h;
     XftColor c;
     GString *text;
-    gint mw, mh;
+    gint mw;
 #ifndef USE_PANGO
+    gint mh;
     size_t l;
     gboolean shortened = FALSE;
 #else

diff --git a/render/font.h b/render/font.h
index f299763946fa89ab7ff6cbc2d2666d1f03188b82..19417db2c1118e92c14da9a5229f715330b99537 100644 (file)
--- a/render/font.h
+++ b/render/font.h
@@ -23,6 +23,7 @@
 #include "geom.h"
 #ifdef USE_PANGO
 #include <pango/pango.h>
+#include <pango/pangoxft.h>
 #endif /* USE_PANGO */
 
 struct _RrFont {

diff --git a/render/image.c b/render/image.c
index 2c32cd127c0fc4ec11ee190272f9e9df627d41a3..8903ad385d79a9368fa344ce1bd71ceec98f5d62 100644 (file)
--- a/render/image.c
+++ b/render/image.c
@@ -26,37 +26,6 @@
 #define FLOOR(i)        ((i) & (~0UL << FRACTION))
 #define AVERAGE(a, b)   (((((a) ^ (b)) & 0xfefefefeL) >> 1) + ((a) & (b)))
 
-static RrPixel32* scale_half(RrPixel32 *source, gint w, gint h)
-{
-    RrPixel32 *out, *dest, *sourceline, *sourceline2;
-    gint dw, dh, x, y;
-
-    sourceline = source;
-    sourceline2 = source + w;
-
-    dw = w >> 1;
-    dh = h >> 1;
-
-    out = dest = g_new(RrPixel32, dw * dh);
-
-    for (y = 0; y < dh; ++y) {
-        RrPixel32 *s, *s2;
-
-        s = sourceline;
-        s2 = sourceline2;
-
-        for (x = 0; x < dw; ++x) {
-            *dest++ = AVERAGE(AVERAGE(*s, *(s+1)),
-                              AVERAGE(*s2, *(s2+1)));
-            s += 2;
-            s2 += 2;
-        }
-        sourceline += w << 1;
-        sourceline2 += w << 1;
-    }
-    return out;
-}
-
 static void ImageCopyResampled(RrPixel32 *dst, RrPixel32 *src,
                                gulong dstW, gulong dstH,
                                gulong srcW, gulong srcH)

diff --git a/render/render.c b/render/render.c
index e97f78559a7a21161a23b59d2e6dcb909cd4cc58..5b6cb95feddddb43b0c5f0b0957b9028c6b1f238 100644 (file)
--- a/render/render.c
+++ b/render/render.c
@@ -375,9 +375,9 @@ void RrMinsize(RrAppearance *a, gint *w, gint *h)
 static void reverse_bits(gchar *c, gint n)
 {
     gint i;
-    for (i = 0; i < n; i++)
-        *c++ = (((*c * 0x0802UL & 0x22110UL) |
-                 (*c * 0x8020UL & 0x88440UL)) * 0x10101UL) >> 16;
+    for (i = 0; i < n; i++, *c++)
+        *c = (((*c * 0x0802UL & 0x22110UL) |
+               (*c * 0x8020UL & 0x88440UL)) * 0x10101UL) >> 16;
 }
 
 gboolean RrPixmapToRGBA(const RrInstance *inst,