+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
/* 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 */
{
gchar buf[128];
gint res;
+ const gchar *fmt;
/* %d basic formatting */
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