headers inclusion is a real mess...
authorcaro <caro@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Fri, 6 Nov 2009 07:50:50 +0000 (07:50 +0000)
committercaro <caro@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Fri, 6 Nov 2009 07:50:50 +0000 (07:50 +0000)
 * headers inclusion should be clean in dbus and hal,
   not that clean in nm. I didn't touch much connman
   as Gustavo seems to rewrite it
 * remove unused param warnings in dbus, hal and nm
 * fix connman compilation (error stuff)

git-svn-id: svn+ssh://svn.enlightenment.org/var/svn/e/trunk/e_dbus@43480 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33

34 files changed:
INSTALL
Makefile.am
configure.ac
src/bin/hal.c
src/bin/nm.c
src/bin/notification_daemon.c
src/bin/notify-send.c
src/bin/notify.c
src/bin/test.c
src/bin/test_client.c
src/lib/connman/E_Connman.h
src/lib/connman/e_connman.c
src/lib/connman/e_connman_private.h
src/lib/dbus/E_DBus.h
src/lib/dbus/Makefile.am
src/lib/dbus/e_dbus.c
src/lib/dbus/e_dbus_interfaces.c
src/lib/dbus/e_dbus_message.c
src/lib/dbus/e_dbus_methods.c
src/lib/dbus/e_dbus_object.c
src/lib/dbus/e_dbus_private.h
src/lib/dbus/e_dbus_signal.c
src/lib/dbus/e_dbus_util.c
src/lib/nm/e_nm.c
src/lib/nm/e_nm_active_connection.c
src/lib/nm/e_nm_device.c
src/lib/nm/e_nm_ip4_config.c
src/lib/nm/e_nm_manager.c
src/lib/nm/e_nm_util.c
src/lib/nm/e_nm_variant.c
src/lib/nm/e_nms.c
src/lib/nm/e_nms_connection.c
src/lib/notification/daemon.c
src/lib/notification/marshal.c

diff --git a/INSTALL b/INSTALL
index 23e5f25..2550dab 100644 (file)
--- a/INSTALL
+++ b/INSTALL
@@ -1,16 +1,19 @@
 Installation Instructions
 *************************
 
-Copyright (C) 1994, 1995, 1996, 1999, 2000, 2001, 2002, 2004, 2005 Free
-Software Foundation, Inc.
+Copyright (C) 1994, 1995, 1996, 1999, 2000, 2001, 2002, 2004, 2005,
+2006, 2007, 2008, 2009 Free Software Foundation, Inc.
 
-This file is free documentation; the Free Software Foundation gives
+   This file is free documentation; the Free Software Foundation gives
 unlimited permission to copy, distribute and modify it.
 
 Basic Installation
 ==================
 
-These are generic installation instructions.
+   Briefly, the shell commands `./configure; make; make install' should
+configure, build, and install this package.  The following
+more-detailed instructions are generic; see the `README' file for
+instructions specific to this package.
 
    The `configure' shell script attempts to guess correct values for
 various system-dependent variables used during compilation.  It uses
@@ -23,9 +26,9 @@ debugging `configure').
 
    It can also use an optional file (typically called `config.cache'
 and enabled with `--cache-file=config.cache' or simply `-C') that saves
-the results of its tests to speed up reconfiguring.  (Caching is
+the results of its tests to speed up reconfiguring.  Caching is
 disabled by default to prevent problems with accidental use of stale
-cache files.)
+cache files.
 
    If you need to do unusual things to compile the package, please try
 to figure out how `configure' could check whether to do them, and mail
@@ -35,20 +38,17 @@ some point `config.cache' contains results you don't want to keep, you
 may remove or edit it.
 
    The file `configure.ac' (or `configure.in') is used to create
-`configure' by a program called `autoconf'.  You only need
-`configure.ac' if you want to change it or regenerate `configure' using
-a newer version of `autoconf'.
+`configure' by a program called `autoconf'.  You need `configure.ac' if
+you want to change it or regenerate `configure' using a newer version
+of `autoconf'.
 
 The simplest way to compile this package is:
 
   1. `cd' to the directory containing the package's source code and type
-     `./configure' to configure the package for your system.  If you're
-     using `csh' on an old version of System V, you might need to type
-     `sh ./configure' instead to prevent `csh' from trying to execute
-     `configure' itself.
+     `./configure' to configure the package for your system.
 
-     Running `configure' takes awhile.  While running, it prints some
-     messages telling which features it is checking for.
+     Running `configure' might take a while.  While running, it prints
+     some messages telling which features it is checking for.
 
   2. Type `make' to compile the package.
 
@@ -67,42 +67,57 @@ The simplest way to compile this package is:
      all sorts of other programs in order to regenerate files that came
      with the distribution.
 
+  6. Often, you can also type `make uninstall' to remove the installed
+     files again.
+
 Compilers and Options
 =====================
 
-Some systems require unusual options for compilation or linking that the
-`configure' script does not know about.  Run `./configure --help' for
-details on some of the pertinent environment variables.
+   Some systems require unusual options for compilation or linking that
+the `configure' script does not know about.  Run `./configure --help'
+for details on some of the pertinent environment variables.
 
    You can give `configure' initial values for configuration parameters
 by setting variables in the command line or in the environment.  Here
 is an example:
 
-     ./configure CC=c89 CFLAGS=-O2 LIBS=-lposix
+     ./configure CC=c99 CFLAGS=-g LIBS=-lposix
 
    *Note Defining Variables::, for more details.
 
 Compiling For Multiple Architectures
 ====================================
 
-You can compile the package for more than one kind of computer at the
+   You can compile the package for more than one kind of computer at the
 same time, by placing the object files for each architecture in their
-own directory.  To do this, you must use a version of `make' that
-supports the `VPATH' variable, such as GNU `make'.  `cd' to the
+own directory.  To do this, you can use GNU `make'.  `cd' to the
 directory where you want the object files and executables to go and run
 the `configure' script.  `configure' automatically checks for the
 source code in the directory that `configure' is in and in `..'.
 
-   If you have to use a `make' that does not support the `VPATH'
-variable, you have to compile the package for one architecture at a
-time in the source code directory.  After you have installed the
-package for one architecture, use `make distclean' before reconfiguring
-for another architecture.
+   With a non-GNU `make', it is safer to compile the package for one
+architecture at a time in the source code directory.  After you have
+installed the package for one architecture, use `make distclean' before
+reconfiguring for another architecture.
+
+   On MacOS X 10.5 and later systems, you can create libraries and
+executables that work on multiple system types--known as "fat" or
+"universal" binaries--by specifying multiple `-arch' options to the
+compiler but only a single `-arch' option to the preprocessor.  Like
+this:
+
+     ./configure CC="gcc -arch i386 -arch x86_64 -arch ppc -arch ppc64" \
+                 CXX="g++ -arch i386 -arch x86_64 -arch ppc -arch ppc64" \
+                 CPP="gcc -E" CXXCPP="g++ -E"
+
+   This is not guaranteed to produce working output in all cases, you
+may have to build one architecture at a time and combine the results
+using the `lipo' tool if you have problems.
 
 Installation Names
 ==================
 
-By default, `make install' installs the package's commands under
+   By default, `make install' installs the package's commands under
 `/usr/local/bin', include files under `/usr/local/include', etc.  You
 can specify an installation prefix other than `/usr/local' by giving
 `configure' the option `--prefix=PREFIX'.
@@ -125,7 +140,7 @@ option `--program-prefix=PREFIX' or `--program-suffix=SUFFIX'.
 Optional Features
 =================
 
-Some packages pay attention to `--enable-FEATURE' options to
+   Some packages pay attention to `--enable-FEATURE' options to
 `configure', where FEATURE indicates an optional part of the package.
 They may also pay attention to `--with-PACKAGE' options, where PACKAGE
 is something like `gnu-as' or `x' (for the X Window System).  The
@@ -137,14 +152,46 @@ find the X include and library files automatically, but if it doesn't,
 you can use the `configure' options `--x-includes=DIR' and
 `--x-libraries=DIR' to specify their locations.
 
+Particular systems
+==================
+
+   On HP-UX, the default C compiler is not ANSI C compatible.  If GNU
+CC is not installed, it is recommended to use the following options in
+order to use an ANSI C compiler:
+
+     ./configure CC="cc -Ae -D_XOPEN_SOURCE=500"
+
+and if that doesn't work, install pre-built binaries of GCC for HP-UX.
+
+   On OSF/1 a.k.a. Tru64, some versions of the default C compiler cannot
+parse its `<wchar.h>' header file.  The option `-nodtk' can be used as
+a workaround.  If GNU CC is not installed, it is therefore recommended
+to try
+
+     ./configure CC="cc"
+
+and if that doesn't work, try
+
+     ./configure CC="cc -nodtk"
+
+   On Solaris, don't put `/usr/ucb' early in your `PATH'.  This
+directory contains several dysfunctional programs; working variants of
+these programs are available in `/usr/bin'.  So, if you need `/usr/ucb'
+in your `PATH', put it _after_ `/usr/bin'.
+
+   On Haiku, software installed for all users goes in `/boot/common',
+not `/usr/local'.  It is recommended to use the following options:
+
+     ./configure --prefix=/boot/common
+
 Specifying the System Type
 ==========================
 
-There may be some features `configure' cannot figure out automatically,
-but needs to determine by the type of machine the package will run on.
-Usually, assuming the package is built to be run on the _same_
-architectures, `configure' can figure that out, but if it prints a
-message saying it cannot guess the machine type, give it the
+   There may be some features `configure' cannot figure out
+automatically, but needs to determine by the type of machine the package
+will run on.  Usually, assuming the package is built to be run on the
+_same_ architectures, `configure' can figure that out, but if it prints
+message saying it cannot guess the machine type, give it the
 `--build=TYPE' option.  TYPE can either be a short name for the system
 type, such as `sun4', or a canonical name which has the form:
 
@@ -152,7 +199,8 @@ type, such as `sun4', or a canonical name which has the form:
 
 where SYSTEM can have one of these forms:
 
-     OS KERNEL-OS
+     OS
+     KERNEL-OS
 
    See the file `config.sub' for the possible values of each field.  If
 `config.sub' isn't included in this package, then this package doesn't
@@ -170,9 +218,9 @@ eventually be run) with `--host=TYPE'.
 Sharing Defaults
 ================
 
-If you want to set default values for `configure' scripts to share, you
-can create a site shell script called `config.site' that gives default
-values for variables like `CC', `cache_file', and `prefix'.
+   If you want to set default values for `configure' scripts to share,
+you can create a site shell script called `config.site' that gives
+default values for variables like `CC', `cache_file', and `prefix'.
 `configure' looks for `PREFIX/share/config.site' if it exists, then
 `PREFIX/etc/config.site' if it exists.  Or, you can set the
 `CONFIG_SITE' environment variable to the location of the site script.
@@ -181,7 +229,7 @@ A warning: not all `configure' scripts look for a site script.
 Defining Variables
 ==================
 
-Variables not defined in a site shell script can be set in the
+   Variables not defined in a site shell script can be set in the
 environment passed to `configure'.  However, some packages may run
 configure again during the build, and the customized values of these
 variables may be lost.  In order to avoid this problem, you should set
@@ -190,21 +238,29 @@ them in the `configure' command line, using `VAR=value'.  For example:
      ./configure CC=/usr/local2/bin/gcc
 
 causes the specified `gcc' to be used as the C compiler (unless it is
-overridden in the site shell script).  Here is a another example:
+overridden in the site shell script).
 
-     /bin/bash ./configure CONFIG_SHELL=/bin/bash
+Unfortunately, this technique does not work for `CONFIG_SHELL' due to
+an Autoconf bug.  Until the bug is fixed you can use this workaround:
 
-Here the `CONFIG_SHELL=/bin/bash' operand causes subsequent
-configuration-related scripts to be executed by `/bin/bash'.
+     CONFIG_SHELL=/bin/bash /bin/bash ./configure CONFIG_SHELL=/bin/bash
 
 `configure' Invocation
 ======================
 
-`configure' recognizes the following options to control how it operates.
+   `configure' recognizes the following options to control how it
+operates.
 
 `--help'
 `-h'
-     Print a summary of the options to `configure', and exit.
+     Print a summary of all of the options to `configure', and exit.
+
+`--help=short'
+`--help=recursive'
+     Print a summary of the options unique to this package's
+     `configure', and exit.  The `short' variant lists options used
+     only in the top level, while the `recursive' variant lists options
+     also present in any nested packages.
 
 `--version'
 `-V'
@@ -231,6 +287,16 @@ configuration-related scripts to be executed by `/bin/bash'.
      Look for the package's source code in directory DIR.  Usually
      `configure' can determine that directory automatically.
 
+`--prefix=DIR'
+     Use DIR as the installation prefix.  *Note Installation Names::
+     for more details, including other options available for fine-tuning
+     the installation locations.
+
+`--no-create'
+`-n'
+     Run the configure checks, but stop before creating any output
+     files.
+
 `configure' also accepts some other, not widely useful, options.  Run
 `configure --help' for more details.
 
index b33aad2..ab20809 100644 (file)
@@ -1,3 +1,5 @@
+ACLOCAL_AMFLAGS = -I m4
+
 SUBDIRS = src
 
 MAINTAINERCLEANFILES = Makefile.in aclocal.m4 config.guess \
index b571fc9..e1923f7 100644 (file)
@@ -4,6 +4,7 @@ rm -f config.cache
 AC_INIT([e_dbus], [0.5.0.062], [enlightenment-devel@lists.sourceforge.net])
 AC_PREREQ([2.52])
 AC_CONFIG_SRCDIR([configure.ac])
+AC_CONFIG_MACRO_DIR([m4])
 AC_CANONICAL_BUILD
 AC_CANONICAL_HOST
 
@@ -14,7 +15,9 @@ m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])])
 AC_PROG_CC
 AC_HEADER_STDC
 AM_PROG_CC_C_O
+AC_C___ATTRIBUTE__
 
+AC_LIBTOOL_WIN32_DLL
 define([AC_LIBTOOL_LANG_CXX_CONFIG], [:])dnl
 define([AC_LIBTOOL_LANG_F77_CONFIG], [:])dnl
 AC_PROG_LIBTOOL
@@ -38,26 +41,25 @@ AC_SUBST(enm_release_info)
 AC_SUBST(enotify_release_info)
 
 dnl Check enabled modules to build
-AC_ARG_ENABLE(ehal,
-                                 AC_HELP_STRING([--disable-ehal],
-                                                                         [Disable ehal build]),
-                                 [enable_ehal=$enableval],
-                                 [enable_ehal="yes"])
-AC_ARG_ENABLE(enm,
-                                 AC_HELP_STRING([--enable-enm],
-                                                                         [Enable enm (network manager) build]),
-                                 [enable_enm=$enableval],
-                                 [enable_enm="no"])
-AC_ARG_ENABLE(econnman,
-                                 AC_HELP_STRING([--enable-econnman],
-                                                                         [Enable econnman build]),
-                                 [enable_econnman=$enableval],
-                                 [enable_econnman="no"])
-AC_ARG_ENABLE(enotify,
-                                 AC_HELP_STRING([--disable-enotify],
-                                                                         [Disable enotify build]),
-                                 [enable_enotify=$enableval],
-                                 [enable_enotify="yes"])
+AC_ARG_ENABLE([ehal],
+   [AC_HELP_STRING([--disable-ehal], [Disable ehal build])],
+   [enable_ehal=$enableval],
+   [enable_ehal="yes"])
+
+AC_ARG_ENABLE([enm],
+   [AC_HELP_STRING([--enable-enm], [Enable enm (network manager) build])],
+   [enable_enm=$enableval],
+   [enable_enm="no"])
+
+AC_ARG_ENABLE([econnman],
+   [AC_HELP_STRING([--enable-econnman], [Enable econnman build])],
+   [enable_econnman=$enableval],
+   [enable_econnman="no"])
+
+AC_ARG_ENABLE([enotify],
+   [AC_HELP_STRING([--disable-enotify], [Disable enotify build])],
+   [enable_enotify=$enableval],
+   [enable_enotify="yes"])
 
 # pkg-config
 PKG_PROG_PKG_CONFIG
@@ -79,29 +81,35 @@ PKG_CHECK_MODULES([EDBUS], [
 
 
 dnl evas
-if test "x${enable_enotify}" = "xyes"
-then
-       PKG_CHECK_MODULES(EVAS, [evas >= 0.9.9], [], [enable_enotify="no"])
+if test "x${enable_enotify}" = "xyes" ; then
+   PKG_CHECK_MODULES(EVAS, [evas >= 0.9.9], [], [enable_enotify="no"])
 fi
 
-AC_ARG_ENABLE(build-test-gui,
-  AS_HELP_STRING([--enable-build-test-gui], [Build with gui for testing]),
-  [ PKG_CHECK_MODULES([HAL_TEST_GUI], [
-    ewl
-    efreet
-    ],
+AC_ARG_ENABLE([build-test-gui],
+   [AS_HELP_STRING([--enable-build-test-gui], [Build with gui for testing])],
+   [PKG_CHECK_MODULES([HAL_TEST_GUI],
+       [ewl efreet],
     # temporarily disable until update to newer tree2 api
     [AC_DEFINE([EWL_GUI], 0, [An ewl gui for e_dbus_hal])],
-    [AC_DEFINE([EWL_GUI], 0, [An ewl gui for e_dbus_hal])]
-    )
- ]
-)
+    [AC_DEFINE([EWL_GUI], 0, [An ewl gui for e_dbus_hal])])
+   ])
 
 AM_CONDITIONAL([BUILD_EHAL],     [test "x${enable_ehal}"     = "xyes"])
 AM_CONDITIONAL([BUILD_ENM],      [test "x${enable_enm}"      = "xyes"])
 AM_CONDITIONAL([BUILD_ENOTIFY],  [test "x${enable_enotify}"  = "xyes"])
 AM_CONDITIONAL([BUILD_ECONNMAN], [test "x${enable_econnman}" = "xyes"])
 
+EFL_EDBUS_BUILD=""
+lt_enable_auto_import=""
+case "$host_os" in
+   mingw* | cegcc*)
+      EFL_EDBUS_BUILD="-DEFL_EDBUS_BUILD"
+      lt_enable_auto_import="-Wl,--enable-auto-import"
+   ;;
+esac
+AC_SUBST(EFL_EDBUS_BUILD)
+AC_SUBST(lt_enable_auto_import)
+
 AC_OUTPUT([
 e_dbus.spec
 Makefile
index d1876ff..61c839e 100644 (file)
@@ -1,13 +1,17 @@
-#include "E_Hal.h"
-#include <config.h>
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <string.h>
+
+#include <Ecore.h>
+#include <E_Hal.h>
 
 #if EWL_GUI
 #include <Ewl.h>
 #include <Efreet.h>
 #endif
 
-#include <string.h>
-
 /* test app */
 
 static E_DBus_Connection *conn;
@@ -376,7 +380,7 @@ volume_append(const char *udi)
 }
 
 static void
-cb_test_get_all_devices(void *user_data, void *reply_data, DBusError *error)
+cb_test_get_all_devices(void *user_data __UNUSED__, void *reply_data, DBusError *error)
 {
   E_Hal_Manager_Get_All_Devices_Return *ret = reply_data;
   Eina_List *l;
@@ -398,7 +402,7 @@ cb_test_get_all_devices(void *user_data, void *reply_data, DBusError *error)
 }
 
 static void
-cb_test_find_device_by_capability_storage(void *user_data, void *reply_data, DBusError *error)
+cb_test_find_device_by_capability_storage(void *user_data __UNUSED__, void *reply_data, DBusError *error)
 {
   E_Hal_Manager_Find_Device_By_Capability_Return *ret = reply_data;
   Eina_List *l;
@@ -418,7 +422,7 @@ cb_test_find_device_by_capability_storage(void *user_data, void *reply_data, DBu
 }
 
 static void
-cb_test_find_device_by_capability_volume(void *user_data, void *reply_data, DBusError *error)
+cb_test_find_device_by_capability_volume(void *user_data __UNUSED__, void *reply_data, DBusError *error)
 {
   E_Hal_Manager_Find_Device_By_Capability_Return *ret = reply_data;
   Eina_List *l;
@@ -478,7 +482,7 @@ error:
 }
 
 static void
-cb_signal_device_added(void *data, DBusMessage *msg)
+cb_signal_device_added(void *data __UNUSED__, DBusMessage *msg)
 {
   DBusError err;
   char *udi;
@@ -493,7 +497,7 @@ cb_signal_device_added(void *data, DBusMessage *msg)
 }
 
 void
-cb_signal_device_removed(void *data, DBusMessage *msg)
+cb_signal_device_removed(void *data __UNUSED__, DBusMessage *msg)
 {
   DBusError err;
   char *udi;
@@ -507,7 +511,7 @@ cb_signal_device_removed(void *data, DBusMessage *msg)
 }
 
 void
-cb_signal_new_capability(void *data, DBusMessage *msg)
+cb_signal_new_capability(void *data __UNUSED__, DBusMessage *msg)
 {
   DBusError err;
   char *udi, *capability;
@@ -782,7 +786,7 @@ mountbox_mainwin_new(void)
 }
 #endif
 int 
-main(int argc, char **argv)
+main()
 {
 #if EWL_GUI
   Ewl_Widget *win;
index 0267ad4..9732743 100644 (file)
@@ -1,8 +1,15 @@
-#include <E_Nm.h>
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
 
+#include <stdio.h>
 #include <unistd.h>
 #include <string.h>
 
+#include <Ecore.h>
+
+#include <E_Nm.h>
+
 E_NM *nm = NULL;
 E_NMS *nms = NULL;
 
@@ -57,7 +64,7 @@ dump_variant(E_NM_Variant *var)
 }
 
 static Eina_Bool
-dump_values(const Eina_Hash *hash, const void *key, void *value, void *data)
+dump_values(const Eina_Hash *hash __UNUSED__, const void *key, void *value, void *data __UNUSED__)
 {
     printf(" - name: %s - ", (char *)key);
     dump_variant(value);
@@ -67,7 +74,7 @@ dump_values(const Eina_Hash *hash, const void *key, void *value, void *data)
 }
  
 static Eina_Bool
-dump_settings(const Eina_Hash *hash, const void *key, void *value, void *fdata)
+dump_settings(const Eina_Hash *hash __UNUSED__, const void *key, void *value, void *fdata __UNUSED__)
 {
     printf("name: %s\n", (char *)key);
     printf("values:\n");
@@ -78,7 +85,7 @@ dump_settings(const Eina_Hash *hash, const void *key, void *value, void *fdata)
 }
 
 static int
-cb_nms_connection_secrets(void *data, Eina_Hash *secrets)
+cb_nms_connection_secrets(void *data __UNUSED__, Eina_Hash *secrets)
 {
     printf("Secrets:\n");
     if (secrets)
@@ -101,7 +108,7 @@ cb_nms_connection_settings(void *data, Eina_Hash *settings)
 }
 
 static int
-cb_nms_connections(void *data, Eina_List *list)
+cb_nms_connections(void *data __UNUSED__, Eina_List *list)
 {
     E_NMS_Connection *conn;
     Eina_List        *l;
@@ -117,7 +124,7 @@ cb_nms_connections(void *data, Eina_List *list)
 }
 
 static int
-cb_access_point(void *data, E_NM_Access_Point *ap)
+cb_access_point(void *data __UNUSED__, E_NM_Access_Point *ap)
 {
     e_nm_access_point_dump(ap);
     e_nm_access_point_free(ap);
@@ -137,7 +144,7 @@ cb_activate_connection(void *data, E_NM_Device *device)
 }
 
 static int
-cb_active_connection(void *data, E_NM_Active_Connection *conn)
+cb_active_connection(void *data __UNUSED__, E_NM_Active_Connection *conn)
 {
     const char *device;
     Eina_List  *l;
@@ -153,7 +160,7 @@ cb_active_connection(void *data, E_NM_Active_Connection *conn)
 }
 
 static int
-cb_ip4_config(void *data, E_NM_IP4_Config *config)
+cb_ip4_config(void *data __UNUSED__, E_NM_IP4_Config *config)
 {
     e_nm_ip4_config_dump(config);
     e_nm_ip4_config_free(config);
@@ -161,10 +168,9 @@ cb_ip4_config(void *data, E_NM_IP4_Config *config)
 }
 
 static int
-cb_access_points(void *data, Eina_List *list)
+cb_access_points(void *data __UNUSED__, Eina_List *list)
 {
     E_NM_Access_Point *ap;
-    Eina_List         *l;
 
     EINA_LIST_FREE(list, ap)
     {
@@ -175,7 +181,7 @@ cb_access_points(void *data, Eina_List *list)
 }
 
 static int
-cb_get_devices(void *data, Eina_List *list)
+cb_get_devices(void *data __UNUSED__, Eina_List *list)
 {
     E_NM_Device *device;
     Eina_List   *l;
@@ -197,7 +203,7 @@ cb_get_devices(void *data, Eina_List *list)
 }
 
 static int
-cb_nms(void *data, E_NMS *reply)
+cb_nms(void *data __UNUSED__, E_NMS *reply)
 {
     Eina_Hash *settings, *values;
     E_NM_Variant variant;
@@ -274,7 +280,7 @@ cb_nms(void *data, E_NMS *reply)
 }
 
 static int
-cb_nm(void *data, E_NM *reply)
+cb_nm(void *data __UNUSED__, E_NM *reply)
 {
     if (!reply)
     {
@@ -301,7 +307,7 @@ cb_nm(void *data, E_NM *reply)
    
 
 int 
-main(int argc, char **argv)
+main()
 {
     ecore_init();
     eina_init();
index 3acc21a..ffddc4a 100644 (file)
@@ -1,3 +1,10 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+
+#include <Ecore.h>
 #include <E_Notification_Daemon.h>
 
 typedef struct Daemon_Data Daemon_Data;
@@ -145,7 +152,7 @@ cb_close_notification(E_Notification_Daemon *daemon, unsigned int notification_i
 
 
 int
-main(int argc, char **argv)
+main()
 {
   E_Notification_Daemon *d;
   E_Notification *n;
index 058675b..89be670 100644 (file)
@@ -1,12 +1,15 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include <stdio.h>
-#include <stdlib.h>
 #include <string.h>
-#include <errno.h>
 #include <limits.h>
+#include <errno.h>
 #include <getopt.h>
-#include <E_Notify.h>
 
-#include "config.h"
+#include <Ecore.h>
+#include <E_Notify.h>
 
 void
 version(void)
@@ -57,7 +60,7 @@ read_int_arg(long long *result, const char *name, intmax_t min, intmax_t max)
 }
 
 void 
-send_cb(void *user_data, void *method_return, DBusError *error)
+send_cb(void *user_data __UNUSED__, void *method_return, DBusError *error __UNUSED__)
 {
    E_Notification_Return_Notify *r = method_return;
 
index 49851a1..5b588ef 100644 (file)
@@ -1,8 +1,15 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+
+#include <Ecore.h>
 #include <E_DBus.h>
 #include <E_Notify.h>
 
 void
-cb_sent(void *data, void *ret, DBusError *err)
+cb_sent(void *data __UNUSED__, void *ret, DBusError *err)
 {
   E_Notification_Return_Notify *notify;
   notify = ret;
@@ -19,7 +26,7 @@ cb_sent(void *data, void *ret, DBusError *err)
 }
 
 int
-cb_timer(void *data)
+cb_timer(void *data __UNUSED__)
 {
   E_Notification *n;
   char buf[1024];
@@ -40,7 +47,7 @@ cb_timer(void *data)
 }
 
 void
-cb_action_invoked(void *data, int type, void *event)
+cb_action_invoked(void *data __UNUSED__, int type __UNUSED__, void *event)
 {
   E_Notification_Event_Action_Invoked *ev;
 
@@ -50,7 +57,7 @@ cb_action_invoked(void *data, int type, void *event)
 }
 
 void
-cb_note_closed(void *data, int type, void *event)
+cb_note_closed(void *data __UNUSED__, int type __UNUSED__, void *event)
 {
   E_Notification_Event_Notification_Closed *ev;
   static const char *reasons[] = {
@@ -66,7 +73,7 @@ cb_note_closed(void *data, int type, void *event)
 }
 
 int
-main(int argc, char **argv)
+main()
 {
   int ret = 0;
   ecore_init();
index 9b6b7d0..e6a38ea 100644 (file)
@@ -1,9 +1,15 @@
-#include <Ecore.h>
-#include "E_DBus.h"
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include <stdio.h>
 #include <string.h>
 #include <dbus/dbus.h>
 
+#include <Ecore.h>
+
+#include "E_DBus.h"
+
 void
 copy_message(DBusMessageIter *from, DBusMessageIter *to)
 {
@@ -47,7 +53,7 @@ copy_message(DBusMessageIter *from, DBusMessageIter *to)
 }
 
 DBusMessage *
-cb_repeat(E_DBus_Object *obj, DBusMessage *msg)
+cb_repeat(E_DBus_Object *obj __UNUSED__, DBusMessage *msg)
 {
   DBusMessage *reply;
   DBusMessageIter from, to;
@@ -62,7 +68,7 @@ cb_repeat(E_DBus_Object *obj, DBusMessage *msg)
 }
 
 void
-cb_request_name(void *data, DBusMessage *msg, DBusError *err)
+cb_request_name(void *data __UNUSED__, DBusMessage *msg __UNUSED__, DBusError *err __UNUSED__)
 {
   // XXX check that this actually succeeded and handle errors...
   printf("request name\n");
@@ -82,7 +88,7 @@ _setup(E_DBus_Connection *conn)
 }
 
 int
-main (int argc, char ** argv)
+main ()
 {
   E_DBus_Connection *conn;
   ecore_init();
index 3a6b8c8..4709158 100644 (file)
@@ -1,3 +1,10 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+
+#include <Ecore.h>
 #include <E_DBus.h>
 
 #define NUM_LOOPS 10000
@@ -5,7 +12,7 @@
 static dbus_uint32_t msg_num = 0;
 
 void
-cb_reply(void *data, DBusMessage *reply, DBusError *error)
+cb_reply(void *data __UNUSED__, DBusMessage *reply, DBusError *error)
 {
   dbus_uint32_t val;
   if (dbus_error_is_set(error))
@@ -44,7 +51,7 @@ send_message(void *data)
 }
 
 int
-main(int argc, char **argv)
+main()
 {
   E_DBus_Connection *conn;
   int ret = 0;
index 7ae4ae2..387d02b 100644 (file)
@@ -1,12 +1,14 @@
 #ifndef E_CONNMAN_H
 #define E_CONNMAN_H
 
-#include <E_DBus.h>
-#include <Ecore_Data.h>
 #include <stdint.h>
 #include <stdbool.h>
 #include <stdio.h>
 
+#include <Eina.h>
+#include <Ecore.h>
+#include <E_DBus.h>
+
 #ifdef EAPI
 #undef EAPI
 #endif
index 3395678..7790ee1 100644 (file)
@@ -55,6 +55,8 @@ const char *e_connman_prop_wifi_passphrase = NULL;
 const char *e_connman_prop_wifi_security = NULL;
 const char *e_connman_prop_wifi_ssid = NULL;
 
+int _e_dbus_connman_log_dom = -1;
+
 const char *
 e_connman_system_bus_name_get(void)
 {
@@ -225,7 +227,7 @@ e_connman_system_init(E_DBus_Connection *edbus_conn)
 
    if(_e_dbus_connman_log_dom < 0) 
      {
-       E_DBUS_LOG_ERR("E-Dbus connman error : impossible to create a log domain for edbus_connman module");
+       ERR("E-Dbus connman error : impossible to create a log domain for edbus_connman module");
        return -1;
      }
 
@@ -344,7 +346,7 @@ e_connman_system_shutdown(void)
 {
    if (init_count == 0)
      {
-       E_DBUS_LOG_ERR("E-Dbus connman Error: connman system already shut down.");
+       ERR("E-Dbus connman Error: connman system already shut down.");
        return 0;
      }
    init_count--;
index 3190e50..0de7999 100644 (file)
@@ -1,8 +1,9 @@
+#include <stdio.h>
+
 #include <Eina.h>
 #include <eina_safety_checks.h>
 
 #include "E_Connman.h"
-#include <stdio.h>
 
 static const char manager_path[] = "/";
 
@@ -42,6 +43,10 @@ extern const char *e_connman_prop_wifi_security;
 extern const char *e_connman_prop_wifi_ssid;
 extern int _e_dbus_connman_log_dom;
 
+#ifndef EINA_LOG_DEFAULT_COLOR
+#define EINA_LOG_DEFAULT_COLOR EINA_COLOR_CYAN
+#endif
+
 #undef DBG
 #undef INF
 #undef WRN
@@ -58,21 +63,21 @@ __dbus_callback_check_and_init(const char *file, int line, const char *function,
    if (!msg)
      {
        if (err)
-         ERROR(file, function, line,
-               "an error was reported by server: "
-               "name=\"%s\", message=\"%s\"",
-               err->name, err->message);
+         ERR(file, function, line,
+              "an error was reported by server: "
+              "name=\"%s\", message=\"%s\"",
+              err->name, err->message);
        else
-         ERROR(file, function, line,
-               "callback without message arguments!");
+         ERR(file, function, line,
+              "callback without message arguments!");
 
        return 0;
      }
 
    if (!dbus_message_iter_init(msg, itr))
      {
-       ERROR(file, function, line,
-            "could not init iterator.");
+       ERR(file, function, line,
+           "could not init iterator.");
        return 0;
      }
 
@@ -89,9 +94,9 @@ __dbus_iter_type_check(const char *file, int line, const char *function, int typ
    if (type == expected)
      return 1;
 
-   ERROR(file, function, line,
-        "expected type %s (%c) but got %c instead!",
-        expected_name, expected, type);
+   ERR(file, function, line,
+       "expected type %s (%c) but got %c instead!",
+       expected_name, expected, type);
 
    return 0;
 }
index fa71bc9..0ff65c0 100644 (file)
@@ -3,12 +3,11 @@
 
 #define DBUS_API_SUBJECT_TO_CHANGE
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <strings.h>
+/* #include <stdio.h> */
+/* #include <stdlib.h> */
+/* #include <strings.h> */
 
 #include <dbus/dbus.h>
-#include <Ecore.h>
 #include <Eina.h>
 
 #ifdef EAPI
index 04b513a..3ed559d 100644 (file)
@@ -1,23 +1,24 @@
 MAINTAINERCLEANFILES = Makefile.in
 
 AM_CPPFLAGS = \
-@EDBUS_CFLAGS@ @EINA_CFLAGS@
+@EFL_EDBUS_BUILD@ \
+@EDBUS_CFLAGS@ \
+@EINA_CFLAGS@
 
 lib_LTLIBRARIES = libedbus.la
 include_HEADERS = E_DBus.h
 
 libedbus_la_SOURCES = \
-       e_dbus_private.h \
-       e_dbus.c \
-       e_dbus_message.c \
-       e_dbus_methods.c \
-       e_dbus_interfaces.c \
-       e_dbus_object.c \
-       e_dbus_util.c \
-       e_dbus_signal.c
+e_dbus.c \
+e_dbus_message.c \
+e_dbus_methods.c \
+e_dbus_interfaces.c \
+e_dbus_object.c \
+e_dbus_util.c \
+e_dbus_signal.c
 
 
-libedbus_la_LIBADD = \
-@EDBUS_LIBS@ @EINA_LIBS@
-libedbus_la_LDFLAGS      = -version-info @version_info@ @edbus_release_info@
+libedbus_la_LIBADD = @EDBUS_LIBS@ @EINA_LIBS@
+libedbus_la_LDFLAGS = -no-undefined @lt_enable_auto_import@ -version-info @version_info@ @edbus_release_info@
 
+EXTRA_DIST = e_dbus_private.h
\ No newline at end of file
index e22b005..6607cf9 100644 (file)
@@ -1,14 +1,12 @@
-#include "E_DBus.h"
-#include "e_dbus_private.h"
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
 
-#include <stdbool.h>
-#include <unistd.h>
 #include <stdio.h>
-#include <stdlib.h>
 #include <string.h>
+#include <stdbool.h>
 
-#include <Ecore.h>
-#include <Ecore_Data.h>
+#include "e_dbus_private.h"
 
 #define NUM_BUS_TYPES 3
 
@@ -204,7 +202,7 @@ cb_main_wakeup(void *data)
 }
 
 static void
-cb_dispatch_status(DBusConnection *conn, DBusDispatchStatus new_status, void *data)
+cb_dispatch_status(DBusConnection *conn __UNUSED__, DBusDispatchStatus new_status, void *data)
 {
   E_DBus_Connection *cd;
 
@@ -279,7 +277,7 @@ cb_timeout_add(DBusTimeout *timeout, void *data)
 }
 
 static void
-cb_timeout_del(DBusTimeout *timeout, void *data)
+cb_timeout_del(DBusTimeout *timeout, void *data __UNUSED__)
 {
   E_DBus_Timeout_Data *td;
   DBG("timeout del!");
@@ -297,7 +295,7 @@ cb_timeout_del(DBusTimeout *timeout, void *data)
 }
 
 static void
-cb_timeout_toggle(DBusTimeout *timeout, void *data)
+cb_timeout_toggle(DBusTimeout *timeout, void *data __UNUSED__)
 {
   E_DBus_Timeout_Data *td;
   DBG("timeout toggle!");
@@ -331,7 +329,7 @@ cb_watch_add(DBusWatch *watch, void *data)
 }
 
 static void
-cb_watch_del(DBusWatch *watch, void *data)
+cb_watch_del(DBusWatch *watch, void *data __UNUSED__)
 {
   E_DBus_Handler_Data *hd;
 
@@ -347,7 +345,7 @@ cb_watch_del(DBusWatch *watch, void *data)
 }
 
 static void
-cb_watch_toggle(DBusWatch *watch, void *data)
+cb_watch_toggle(DBusWatch *watch, void *data __UNUSED__)
 {
   E_DBus_Handler_Data *hd;
 
@@ -362,13 +360,13 @@ cb_watch_toggle(DBusWatch *watch, void *data)
 }
 
 static void
-e_dbus_message_free(void *data, void *message)
+e_dbus_message_free(void *data __UNUSED__, void *message)
 {
   dbus_message_unref(message);
 }
 
 static DBusHandlerResult
-e_dbus_filter(DBusConnection *conn, DBusMessage *message, void *user_data)
+e_dbus_filter(DBusConnection *conn __UNUSED__, DBusMessage *message, void *user_data)
 {
   E_DBus_Connection *cd = user_data;
   DBG("-----------------");
index 1da46e9..5cdb749 100644 (file)
@@ -1,4 +1,7 @@
-#include "E_DBus.h"
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include "e_dbus_private.h"
 
 /**
index bc42498..9331996 100644 (file)
@@ -1,5 +1,7 @@
-#include <stdlib.h>
-#include "E_DBus.h"
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include "e_dbus_private.h"
 
 typedef struct E_DBus_Pending_Call_Data E_DBus_Pending_Call_Data;
index 71250b6..636555e 100644 (file)
@@ -1,4 +1,7 @@
-#include "E_DBus.h"
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include "e_dbus_private.h"
 
 static inline DBusMessage *
index 102b141..399cb9a 100644 (file)
@@ -1,10 +1,13 @@
-#include "E_DBus.h"
-#include "e_dbus_private.h"
-#include <Eina.h>
-#include <stdio.h>
-#include <stdlib.h>
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include <string.h>
 
+#include <Ecore_Data.h>
+
+#include "e_dbus_private.h"
+
 static E_DBus_Interface *introspectable_interface = NULL;
 static E_DBus_Interface *properties_interface = NULL;
 
@@ -554,7 +557,7 @@ e_dbus_object_handler(DBusConnection *conn, DBusMessage *message, void *user_dat
 }
 
 static void
-e_dbus_object_unregister(DBusConnection *conn, void *user_data)
+e_dbus_object_unregister(DBusConnection *conn __UNUSED__, void *user_data __UNUSED__)
 {
   /* free up the object struct? */
 }
index 67b0e0a..7522ed3 100644 (file)
@@ -1,9 +1,12 @@
 #ifndef E_DBUS_PRIVATE_H
 #define E_DBUS_PRIVATE_H
 
-#include "dbus/dbus.h"
-#include "Ecore_Data.h"
-#include "eina_log.h"
+#include <Eina.h>
+#include <Ecore.h>
+
+#include <dbus/dbus.h>
+
+#include "E_DBus.h"
 
 #ifndef E_DBUS_COLOR_DEFAULT
 #define E_DBUS_COLOR_DEFAULT EINA_COLOR_CYAN
index fe3cc34..567e5f5 100644 (file)
@@ -1,11 +1,10 @@
-#include <stdlib.h>
-#include <stdio.h>
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include <string.h>
-#include <Ecore.h>
-#include <Ecore_Data.h>
-#include "E_DBus.h"
+
 #include "e_dbus_private.h"
-#include "dbus/dbus.h"
 
 struct E_DBus_Signal_Handler
 {
index 27c08fa..e03c02e 100644 (file)
@@ -1,4 +1,7 @@
-#include "E_DBus.h"
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include "e_dbus_private.h"
 
 /**
index d999c4d..16fc116 100644 (file)
@@ -1,7 +1,13 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+#include <string.h>
+
 #include "E_Nm.h"
 #include "e_nm_private.h"
 #include "e_dbus_private.h"
-#include <string.h>
 
 static const Property properties[] = {
   { .name = "WirelessEnabled", .sig = "b", .offset = offsetof(E_NM, wireless_enabled) },
index 481d340..8a35816 100644 (file)
@@ -1,7 +1,13 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+#include <string.h>
+
 #include "E_Nm.h"
 #include "e_nm_private.h"
 #include "e_dbus_private.h"
-#include <string.h>
 
 static const Property active_connection_properties[] = {
   { .name = "ServiceName", .sig = "s", .offset = offsetof(E_NM_Active_Connection, service_name) },
index f84a939..d962aa5 100644 (file)
@@ -3,10 +3,16 @@
  * the org.freedesktop.NetworkManager.Device DBus interface.
  */
 
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+#include <string.h>
+
 #include "E_Nm.h"
 #include "e_nm_private.h"
 #include "e_dbus_private.h"
-#include <string.h>
 
 static void property_device_type(Property_Data *data, DBusMessageIter *iter);
 
index 9e9c449..8ad3a4e 100644 (file)
@@ -1,7 +1,13 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+#include <string.h>
+
 #include "E_Nm.h"
 #include "e_nm_private.h"
 #include "e_dbus_private.h"
-#include <string.h>
 
 static const Property ip4_config_properties[] = {
   { .name = "Addresses", .sig = "aau", .offset = offsetof(E_NM_IP4_Config, addresses) },
index fceb4f8..12e5e03 100644 (file)
@@ -1,3 +1,9 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <unistd.h>
+
 #include "E_Nm.h"
 #include "e_nm_private.h"
 #include "e_dbus_private.h"
index 9b343c8..c93e610 100644 (file)
@@ -1,8 +1,14 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+#include <string.h>
+
 #include "E_Nm.h"
 #include "e_nm_private.h"
 #include "E_DBus.h"
 #include "e_dbus_private.h"
-#include <string.h>
 
 #define CHECK_SIGNATURE(msg, err, sig)                       \
   if (dbus_error_is_set((err)))                              \
@@ -140,7 +146,7 @@ property_basic(DBusMessageIter *iter, const char *sig, void *value)
 }
 
 static E_NM_Variant *
-property_variant(DBusMessageIter *iter, const char *sig, void *value)
+property_variant(DBusMessageIter *iter, const char *sig __UNUSED__, void *value __UNUSED__)
 {
   DBusMessageIter v_iter;
   Property_Cb     func;
index ffcb074..9d0ea7f 100644 (file)
@@ -2,6 +2,7 @@
 
 #include "E_Nm.h"
 #include "e_dbus_private.h"
+
 EAPI E_NM_Variant *
 e_nm_variant_new(int type, const void *value)
 {
index d2c060b..5d8a637 100644 (file)
@@ -1,7 +1,13 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+#include <string.h>
+
 #include "E_Nm.h"
 #include "e_nm_private.h"
 #include "e_dbus_private.h"
-#include <string.h>
 
 static const Property nms_properties[] = {
   { .name = "UnmanagedDevices", .sig = "ao", .offset = offsetof(E_NMS, unmanaged_devices) },
@@ -137,7 +143,7 @@ add_variant(DBusMessageIter *iter, E_NM_Variant *variant)
 }
 
 static Eina_Bool
-add_value(const Eina_Hash *hash, const void *key, void *data, void *fdata)
+add_value(const Eina_Hash *hash __UNUSED__, const void *key, void *data, void *fdata)
 {
   DBusMessageIter *iter, d_iter;
 
@@ -152,7 +158,7 @@ add_value(const Eina_Hash *hash, const void *key, void *data, void *fdata)
 }
 
 static Eina_Bool
-add_array(const Eina_Hash *hash, const void *key, void *data, void *fdata)
+add_array(const Eina_Hash *hash __UNUSED__, const void *key, void *data, void *fdata)
 {
   DBusMessageIter *iter, d_iter, a_iter;
 
@@ -267,7 +273,7 @@ e_nms_system_save_hostname(E_NMS *nms, const char *hostname)
 }
 
 static void
-cb(void *data, DBusMessage *msg, DBusError *err)
+cb(void *data __UNUSED__, DBusMessage *msg __UNUSED__, DBusError *err)
 {
   if (dbus_error_is_set(err))
   {
index 25a0c5d..7ad662d 100644 (file)
@@ -1,7 +1,13 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <stdio.h>
+#include <string.h>
+
 #include "E_Nm.h"
 #include "e_nm_private.h"
 #include "e_dbus_private.h"
-#include <string.h>
 
 static void
 cb_updated(void *data, DBusMessage *msg)
index fd82524..4278b6c 100644 (file)
@@ -1,3 +1,7 @@
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include "E_Notify.h"
 #include "E_Notification_Daemon.h"
 #include "e_notify_private.h"
@@ -7,7 +11,7 @@ static int e_notification_daemon_bus_init(E_Notification_Daemon *daemon);
 static int e_notification_daemon_object_init(E_Notification_Daemon *daemon);
 
 DBusMessage *
-method_get_capabilities(E_DBus_Object *obj, DBusMessage *message)
+method_get_capabilities(E_DBus_Object *obj __UNUSED__, DBusMessage *message)
 {
   const char *capabilities[] = {
     "body",
index 03844a2..91b9389 100644 (file)
@@ -1,5 +1,10 @@
-#include "E_Notify.h"
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
 #include <string.h>
+
+#include "E_Notify.h"
 #include "e_notify_private.h"
 
 void
@@ -89,7 +94,7 @@ e_notify_marshal_string_list_as_array(DBusMessageIter *iter, Eina_List *strings)
 }
 
 Eina_List *
-e_notify_unmarshal_string_array_as_list(DBusMessageIter *iter, DBusError *err)
+e_notify_unmarshal_string_array_as_list(DBusMessageIter *iter, DBusError *err __UNUSED__)
 {
   Eina_List *strings;
   char *sig;
@@ -355,7 +360,7 @@ e_notify_marshal_notify(E_Notification *n)
 }
 
 E_Notification *
-e_notify_unmarshal_notify(DBusMessage *msg, DBusError *err)
+e_notify_unmarshal_notify(DBusMessage *msg, DBusError *err __UNUSED__)
 {
   E_Notification *n;
   const char *s_val;