gnome-doc-tools version doesn't need to be so high. disable the entire

	* configure.in: gnome-doc-tools version doesn't need to be so high.
	* src/compositor/compositor-xrender.c: disable the entire file if the
          compositor is disabled.
	* src/core/async-getprop.[ch]: fixes for -Wall
	* src/core/iconcache.c: fixes for -Wall
	* src/core/testasyncgetprop.c: fixes for -Wall
	* src/core/xprops.c: fixes for -Wall


svn path=/trunk/; revision=4045

8 files changed, 32 insertions, 16 deletions

diff --git a/ChangeLog b/ChangeLog
index 05efdbd8..c8c6f1d7 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2008-12-01  Thomas Thurman  <tthurman@gnome.org>
+
+	* configure.in: gnome-doc-tools version doesn't need to be so high.
+	* src/compositor/compositor-xrender.c: disable the entire file if the
+          compositor is disabled.
+	* src/core/async-getprop.[ch]: fixes for -Wall
+	* src/core/iconcache.c: fixes for -Wall
+	* src/core/testasyncgetprop.c: fixes for -Wall
+	* src/core/xprops.c: fixes for -Wall
+
 2008-11-26  Thomas Thurman  <tthurman@gnome.org>
 
 	* tools/announce-wrangler.py: linked language codes to po files
diff --git a/configure.in b/configure.in
index 871c5dee..6e7c99a5 100644
--- a/configure.in
+++ b/configure.in
@@ -231,7 +231,7 @@ fi
 
 if test x$have_xcomposite = xyes; then
   METACITY_PC_MODULES="$METACITY_PC_MODULES xcomposite >= $XCOMPOSITE_VERSION xfixes xrender xdamage"
-  AC_DEFINE(HAVE_COMPOSITE_EXTENSIONS, , [Building with compositing manager support])
+  AC_DEFINE(HAVE_COMPOSITE_EXTENSIONS, 1, [Building with compositing manager support])
   echo "Building with compositing manager"
 
   ## force on render also
@@ -487,7 +487,7 @@ fi
 CFLAGS="$CFLAGS -Wall -Werror -ansi"
 
 # Use gnome-doc-utils:
-GNOME_DOC_INIT([0.9.0])
+GNOME_DOC_INIT([0.8.0])
 
 AC_CONFIG_FILES([
 Makefile
diff --git a/src/compositor/compositor-xrender.c b/src/compositor/compositor-xrender.c
index a85448db..8c06d1f7 100644
--- a/src/compositor/compositor-xrender.c
+++ b/src/compositor/compositor-xrender.c
@@ -26,6 +26,8 @@
 
 #include <config.h>
 
+#ifdef HAVE_COMPOSITE_EXTENSIONS
+
 #include <stdlib.h>
 #include <string.h>
 #include <math.h>
@@ -3071,3 +3073,6 @@ meta_compositor_xrender_new (MetaDisplay *display)
   return NULL;
 #endif
 }
+
+#endif /* HAVE_COMPOSITE_EXTENSIONS */
+
diff --git a/src/core/async-getprop.c b/src/core/async-getprop.c
index fb5484c1..31b0edc3 100644
--- a/src/core/async-getprop.c
+++ b/src/core/async-getprop.c
@@ -586,7 +586,7 @@ ag_task_get_reply_and_free (AgGetPropertyTask  *task,
                             int                *actual_format,
                             unsigned long      *nitems,
                             unsigned long      *bytesafter,
-                            char              **prop)
+                            unsigned char     **prop)
 {
   Display *dpy;
 
@@ -615,7 +615,7 @@ ag_task_get_reply_and_free (AgGetPropertyTask  *task,
   *nitems = task->n_items;
   *bytesafter = task->bytes_after;
 
-  *prop = task->data; /* pass out ownership of task->data */
+  *prop = (unsigned char*) task->data; /* pass out ownership of task->data */
 
   SyncHandle ();
 
diff --git a/src/core/async-getprop.h b/src/core/async-getprop.h
index e11ca131..c857e930 100644
--- a/src/core/async-getprop.h
+++ b/src/core/async-getprop.h
@@ -47,7 +47,7 @@ Status             ag_task_get_reply_and_free (AgGetPropertyTask  *task,
                                                int                *actual_format,
                                                unsigned long      *nitems,
                                                unsigned long      *bytesafter,
-                                               char              **prop);
+                                               unsigned char     **prop);
 
 Bool     ag_task_have_reply   (AgGetPropertyTask *task);
 Atom     ag_task_get_property (AgGetPropertyTask *task);
diff --git a/src/core/iconcache.c b/src/core/iconcache.c
index c4fd19f5..084eeee0 100644
--- a/src/core/iconcache.c
+++ b/src/core/iconcache.c
@@ -225,11 +225,12 @@ read_rgb_icon (MetaDisplay   *display,
   gulong nitems;
   gulong bytes_after;
   int result, err;
-  gulong *data;
+  guchar *data;
   gulong *best;
   int w, h;
   gulong *best_mini;
   int mini_w, mini_h;
+  gulong *data_as_long = (gulong*) data;
 
   meta_error_trap_push_with_return (display);
   type = None;
@@ -239,8 +240,7 @@ read_rgb_icon (MetaDisplay   *display,
                                display->atom__NET_WM_ICON,
 			       0, G_MAXLONG,
 			       False, XA_CARDINAL, &type, &format, &nitems,
-			       &bytes_after, ((guchar **)&data));
-
+			       &bytes_after, &data);
   err = meta_error_trap_pop_with_return (display, TRUE);
 
   if (err != Success ||
@@ -253,7 +253,7 @@ read_rgb_icon (MetaDisplay   *display,
       return FALSE;
     }
 
-  if (!find_best_size (data, nitems,
+  if (!find_best_size (data_as_long, nitems,
                        ideal_width, ideal_height,
                        &w, &h, &best))
     {
@@ -261,7 +261,7 @@ read_rgb_icon (MetaDisplay   *display,
       return FALSE;
     }
 
-  if (!find_best_size (data, nitems,
+  if (!find_best_size (data_as_long, nitems,
                        ideal_mini_width, ideal_mini_height,
                        &mini_w, &mini_h, &best_mini))
     {
@@ -464,7 +464,8 @@ get_kwm_win_icon (MetaDisplay *display,
   int format;
   gulong nitems;
   gulong bytes_after;
-  Pixmap *icons;
+  guchar *data;
+  Pixmap *icons = (Pixmap*) icons;
   int err, result;
 
   *pixmap = None;
@@ -478,7 +479,7 @@ get_kwm_win_icon (MetaDisplay *display,
 			       False,
                                display->atom__KWM_WIN_ICON,
 			       &type, &format, &nitems,
-			       &bytes_after, (guchar **)&icons);
+			       &bytes_after, &data);
 
   err = meta_error_trap_pop_with_return (display, TRUE);
   if (err != Success ||
diff --git a/src/core/testasyncgetprop.c b/src/core/testasyncgetprop.c
index 267f16a5..03182680 100644
--- a/src/core/testasyncgetprop.c
+++ b/src/core/testasyncgetprop.c
@@ -187,7 +187,7 @@ try_get_reply (Display           *xdisplay,
       int actual_format;
       unsigned long n_items;
       unsigned long bytes_after;
-      char *data;
+      unsigned char *data;
       char *name;
       struct timeval current_time;
 
@@ -421,7 +421,7 @@ run_speed_comparison (Display *xdisplay,
           int actual_format;
           unsigned long n_items;
           unsigned long bytes_after;
-          char *data;
+          unsigned char *data;
 
           assert (ag_task_have_reply (task));
           
diff --git a/src/core/xprops.c b/src/core/xprops.c
index d7801282..a39c7b4f 100644
--- a/src/core/xprops.c
+++ b/src/core/xprops.c
@@ -199,7 +199,7 @@ get_property (MetaDisplay        *display,
                           False, req_type, &results->type, &results->format,
                           &results->n_items,
                           &results->bytes_after,
-                          (guchar **)&results->prop) != Success ||
+                          &results->prop) != Success ||
       results->type == None)
     {
       if (results->prop)
@@ -1051,7 +1051,7 @@ meta_prop_get_values (MetaDisplay   *display,
                                       &results.type, &results.format,
                                       &results.n_items,
                                       &results.bytes_after,
-                                      (char **)(&results.prop)) != Success ||
+                                      &results.prop) != Success ||
           results.type == None)
         {
           values[i].type = META_PROP_VALUE_INVALID;