add some more warning CFLAGS, inspired by Benjamin Otte's blog post,
authorDan Winship <danw@src.gnome.org>
Tue, 23 Dec 2008 19:05:12 +0000 (19:05 +0000)
committerDan Winship <danw@src.gnome.org>
Tue, 23 Dec 2008 19:05:12 +0000 (19:05 +0000)
commit903af762e3acbbe8861b928a7b940e62e1ba0055
tree41c453386191032349f9aa434aedd0e70a8f0669
parent811527eeeec4147bee99baf3d3633c1e07c73868
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
ChangeLog
configure.in
libsoup/soup-cookie-jar.c
libsoup/soup-message-headers.c
tests/forms-test.c
tests/get.c
tests/header-parsing.c
tests/range-test.c
tests/test-utils.c