add some more warning CFLAGS, inspired by Benjamin Otte's blog post,

	* configure.in: add some more warning CFLAGS, inspired by Benjamin
	Otte's blog post, although none of them picked out any actual
	bugs. Annoyingly, the most interesting warnings came from
	-Wwrite-strings and -Wshadow, both of which I decided against
	keeping, because they had too many false positives.

	* libsoup/soup-cookie-jar.c (soup_cookie_jar_get_cookies): rename
	a variable to avoid shadowing.

	* libsoup/soup-message-headers.c
	(soup_message_headers_get_ranges): move a variable declaration to
	avoid a possibly-confusing shadowing.

	* tests/forms-test.c:
	* tests/header-parsing.c:
	* tests/range-test.c:
	* tests/test-utils.c: constify some "char *"s that should have
	already been const.

	* tests/get.c (find_hrefs): rename an arg whose name shadowed a
	global, to avoid possible future confusion
	(get_url): Likewise with a functional-internal shadowing.

svn path=/trunk/; revision=1222

1 files changed, 3 insertions, 3 deletions

diff --git a/tests/test-utils.c b/tests/test-utils.c
index c5e2918b..bc48a551 100644
--- a/tests/test-utils.c
+++ b/tests/test-utils.c
@@ -148,9 +148,9 @@ debug_printf (int level, const char *format, ...)
 #ifdef HAVE_APACHE
 
 static gboolean
-apache_cmd (char *cmd)
+apache_cmd (const char *cmd)
 {
-	char *argv[8];
+	const char *argv[8];
 	char *cwd, *conf;
 	int status;
 	gboolean ok;
@@ -167,7 +167,7 @@ apache_cmd (char *cmd)
 	argv[6] = cmd;
 	argv[7] = NULL;
 
-	ok = g_spawn_sync (cwd, argv, NULL, 0, NULL, NULL,
+	ok = g_spawn_sync (cwd, (char **)argv, NULL, 0, NULL, NULL,
 			   NULL, NULL, &status, NULL);
 	if (ok)
 		ok = (status == 0);