make this a "const gpointer" rather than a gconstpointer to avoid warnings

	* glib/gthreadpool.c (wakeup_thread_marker): make this a "const
	gpointer" rather than a gconstpointer to avoid warnings later

	* glib/pcre/pcre_ucp_searchfuncs.c:
	* glib/pcre/pcre_valid_utf8.c: #include "config.h"

	* glib/tests/printf.c (test_d): fool gcc into not warning about
	some printf format strings that we know are dubious

svn path=/trunk/; revision=7552

diff --git a/ChangeLog b/ChangeLog
index 62a5cad9a1d0b4d22c3a20cf666db946ea5a670b..25341931f3133b8b5740f479d21d0f370a402a61 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2008-09-26  Dan Winship  <danw@gnome.org>
+
+       * glib/gthreadpool.c (wakeup_thread_marker): make this a "const
+       gpointer" rather than a gconstpointer to avoid warnings later
+
+       * glib/pcre/pcre_ucp_searchfuncs.c:
+       * glib/pcre/pcre_valid_utf8.c: #include "config.h"
+
+       * glib/tests/printf.c (test_d): fool gcc into not warning about
+       some printf format strings that we know are dubious
+
 2008-09-26  Matthias Clasen  <mclasen@redhat.com>
 
        Bug 553857 – gbacktrace.h requires signal.h

diff --git a/glib/gthreadpool.c b/glib/gthreadpool.c
index ec50609b1b6ffc49424c7affbab6b2f560fc8f2c..2059a999dbd9a560095e8bbe19f67bd1e20c4f89 100644 (file)
--- a/glib/gthreadpool.c
+++ b/glib/gthreadpool.c
@@ -51,7 +51,7 @@ struct _GRealThreadPool
 /* The following is just an address to mark the wakeup order for a
  * thread, it could be any address (as long, as it isn't a valid
  * GThreadPool address) */
-static gconstpointer const wakeup_thread_marker = (gconstpointer) &g_thread_pool_new;
+static const gpointer wakeup_thread_marker = (gpointer) &g_thread_pool_new;
 static gint wakeup_thread_serial = 0;
 
 /* Here all unused threads are waiting  */

diff --git a/glib/pcre/pcre_ucp_searchfuncs.c b/glib/pcre/pcre_ucp_searchfuncs.c
index b95d2794ca6a03226c022edc9c8a4b37df0a22b8..77ec8d14b50e6927991666f79f711d28910d2da9 100644 (file)
--- a/glib/pcre/pcre_ucp_searchfuncs.c
+++ b/glib/pcre/pcre_ucp_searchfuncs.c
@@ -43,6 +43,10 @@ POSSIBILITY OF SUCH DAMAGE.
 /* This module contains code for searching the table of Unicode character
 properties. */
 
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include "pcre_internal.h"
 
 #include "ucp.h"               /* Category definitions */

diff --git a/glib/pcre/pcre_valid_utf8.c b/glib/pcre/pcre_valid_utf8.c
index a5766b454d88cab8d15fff871c74f33143ac60fe..b7671a96ea0fc757db97ad20ce505b959461e05c 100644 (file)
--- a/glib/pcre/pcre_valid_utf8.c
+++ b/glib/pcre/pcre_valid_utf8.c
@@ -1,3 +1,4 @@
+#include "config.h"
 #include "pcre_internal.h"
 
 /*

diff --git a/glib/tests/printf.c b/glib/tests/printf.c
index f15ace0a1fa904d496f15df75e7fd54eacb8bc6b..ff8eee275213c40a81712c6af89bc5337c954540 100644 (file)
--- a/glib/tests/printf.c
+++ b/glib/tests/printf.c
@@ -69,6 +69,7 @@ test_d (void)
 {
   gchar buf[128];
   gint res;
+  const gchar *fmt;
 
   /* %d basic formatting */
 
@@ -172,21 +173,28 @@ test_d (void)
   g_assert_cmpint (res, ==, 1);
   g_assert_cmpstr (buf, ==, " ");
 
-  res = g_snprintf (buf, 128, "% +d", 5);
-  g_assert_cmpint (res, ==, 2);
-  g_assert_cmpstr (buf, ==, "+5");
-
   res = g_snprintf (buf, 128, "%03d", 5);
   g_assert_cmpint (res, ==, 3);
   g_assert_cmpstr (buf, ==, "005");
 
-  res = g_snprintf (buf, 128, "%-03d", -5);
-  g_assert_cmpint (res, ==, 3);
-  g_assert_cmpstr (buf, ==, "-5 ");
-
   res = g_snprintf (buf, 128, "%03d", -5);
   g_assert_cmpint (res, ==, 3);
   g_assert_cmpstr (buf, ==, "-05");
+
+  /* gcc emits warnings for the following formats, since the C spec
+   * says some of the flags must be ignored. (The " " in "% +d" and
+   * the "0" in "%-03d".) But we need to test that our printf gets
+   * those rules right. So we fool gcc into not warning.
+   */
+  fmt = "% +d";
+  res = g_snprintf (buf, 128, fmt, 5);
+  g_assert_cmpint (res, ==, 2);
+  g_assert_cmpstr (buf, ==, "+5");
+
+  fmt = "%-03d";
+  res = g_snprintf (buf, 128, fmt, -5);
+  g_assert_cmpint (res, ==, 3);
+  g_assert_cmpstr (buf, ==, "-5 ");
 }
 
 static void