This email list is read-only. Emails sent to this list will be discarded ---------------------------------- acinclude.m4 | 15 +++++++++++++++ configure.ac | 6 ++---- gdbus/object.c | 3 ++- plugins/wifi.c | 2 +- src/dbus.c | 2 +- 5 files changed, 21 insertions(+), 7 deletions(-)
New commits: commit da9561f7d0b8eee43a26321dcf50527615d9655d Author: Marcel Holtmann <[email protected]> Date: Tue Dec 23 06:06:09 2008 +0100 Enable verbose compiler warnings in maintainer mode commit e524dd47092369054fed1990ffb317f8b0b59a86 Author: Marcel Holtmann <[email protected]> Date: Tue Dec 23 03:37:10 2008 +0100 Use an unsigned iterator variable commit 9f806bc18d3ef523c069636ecc28d68137f59786 Author: Marcel Holtmann <[email protected]> Date: Tue Dec 23 03:34:00 2008 +0100 Use global connman.h header file commit 6019cedabb291c6b34dd78a4e89f3f9632c6dd91 Author: Marcel Holtmann <[email protected]> Date: Tue Dec 23 03:31:21 2008 +0100 Fix issue with signed/unsigned comparison Diff in this email is a maximum of 400 lines. diff --git a/acinclude.m4 b/acinclude.m4 index 9c51ddf..84086fd 100644 --- a/acinclude.m4 +++ b/acinclude.m4 @@ -10,6 +10,21 @@ AC_DEFUN([AC_PROG_CC_PIE], [ ]) ]) +AC_DEFUN([COMPILER_FLAGS], [ + if (test "${CFLAGS}" = ""); then + CFLAGS="-Wall -O2 -D_FORTIFY_SOURCE=2" + fi + if (test "$USE_MAINTAINER_MODE" = "yes"); then + CFLAGS+=" -Werror -Wextra" + CFLAGS+=" -Wno-unused-parameter" + CFLAGS+=" -Wno-missing-field-initializers" + CFLAGS+=" -Wdeclaration-after-statement" + CFLAGS+=" -Wmissing-declarations" + CFLAGS+=" -Wredundant-decls" + CFLAGS+=" -Wcast-align" + fi +]) + AC_DEFUN([GTK_DOC_CHECK], [ AC_BEFORE([AC_PROG_LIBTOOL],[$0])dnl setup libtool first diff --git a/configure.ac b/configure.ac index 15acdbc..42789a7 100644 --- a/configure.ac +++ b/configure.ac @@ -8,9 +8,7 @@ AM_MAINTAINER_MODE AC_PREFIX_DEFAULT(/usr/local) -if (test "${CFLAGS}" = ""); then - CFLAGS="-Wall -O2 -D_FORTIFY_SOURCE=2" -fi +COMPILER_FLAGS AC_LANG_C @@ -32,7 +30,7 @@ AC_ARG_ENABLE(debug, AC_HELP_STRING([--enable-debug], [enable compiling with debugging information]), [ if (test "${enableval}" = "yes" && test "${ac_cv_prog_cc_g}" = "yes"); then - CFLAGS="$CFLAGS -g -O0 -Werror" + CFLAGS="$CFLAGS -g -O0" fi ]) diff --git a/gdbus/object.c b/gdbus/object.c index 4375ec5..432a8a0 100644 --- a/gdbus/object.c +++ b/gdbus/object.c @@ -59,7 +59,8 @@ static void print_arguments(GString *gstr, const char *sig, for (i = 0; sig[i]; i++) { char type[32]; - int len, struct_level, dict_level; + int struct_level, dict_level; + unsigned int len; gboolean complete; complete = FALSE; diff --git a/plugins/wifi.c b/plugins/wifi.c index f70f0b6..9a92ad5 100644 --- a/plugins/wifi.c +++ b/plugins/wifi.c @@ -304,7 +304,7 @@ static void scan_result(struct connman_element *device, struct wifi_data *data = connman_element_get_data(device); struct connman_element *element; gchar *temp; - int i; + unsigned int i; DBG("network %p identifier %s", network, network->identifier); diff --git a/src/dbus.c b/src/dbus.c index 6960a40..f985e95 100644 --- a/src/dbus.c +++ b/src/dbus.c @@ -23,7 +23,7 @@ #include <config.h> #endif -#include <connman/dbus.h> +#include "connman.h" void connman_dbus_property_append_variant(DBusMessageIter *iter, const char *key, int type, void *val) _______________________________________________ Commits mailing list [email protected] https://lists.moblin.org/mailman/listinfo/commits
