diff options
author | Samuel Ortiz <sameo@linux.intel.com> | 2012-04-16 13:05:57 +0200 |
---|---|---|
committer | Samuel Ortiz <sameo@linux.intel.com> | 2012-04-16 13:05:57 +0200 |
commit | 8b8171237e9b3f1d0c360299efd1001368a3bf63 (patch) | |
tree | a3c869151dd5130a6a4a46da191aa3f2ac87bea6 | |
parent | 4ddda55162cb3ab1889a41d85b16651364907b1d (diff) | |
download | connman-gnome-8b8171237e9b3f1d0c360299efd1001368a3bf63.tar.gz |
common: Fix warning and build failures
-rw-r--r-- | common/connman-agent.c | 68 | ||||
-rw-r--r-- | common/connman-client.c | 7 | ||||
-rw-r--r-- | common/connman-dbus.c | 9 |
3 files changed, 19 insertions, 65 deletions
diff --git a/common/connman-agent.c b/common/connman-agent.c index ef9ed1c..49ed0c7 100644 --- a/common/connman-agent.c +++ b/common/connman-agent.c @@ -52,22 +52,6 @@ static GQuark agent_error_quark(void) #define ENUM_ENTRY(NAME, DESC) { NAME, "" #NAME "", DESC } -static GType agent_error_get_type(void) -{ - static GType etype = 0; - if (etype == 0) { - static const GEnumValue values[] = { - ENUM_ENTRY(AGENT_ERROR_REJECT, "Rejected"), - ENUM_ENTRY(AGENT_ERROR_RETRY, "Retry"), - { 0, 0, 0 } - }; - - etype = g_enum_register_static("Agent", values); - } - - return etype; -} - typedef struct _ConnmanAgentPrivate ConnmanAgentPrivate; typedef struct _PendingRequest PendingRequest; @@ -146,23 +130,11 @@ gboolean connman_agent_request_input_abort(gpointer request_data) return FALSE; } -static gboolean connman_agent_request_input_cb(const GHashTable *reply, gpointer user_data) -{ - - PendingRequest *pendingrequest = user_data; - - dbus_g_method_return(pendingrequest->context, reply); - - g_free(pendingrequest); - return FALSE; -} - -gboolean connman_agent_report_error(ConnmanAgent *agent, - const char *path, const char *error, - DBusGMethodInvocation *context) +static gboolean connman_agent_report_error(ConnmanAgent *agent, + const char *path, const char *error, + DBusGMethodInvocation *context) { GError *result; - ConnmanAgentPrivate *priv = CONNMAN_AGENT_GET_PRIVATE(agent); debug(agent, "connection %s, reports an error: %s", path, error); result = g_error_new(AGENT_ERROR, AGENT_ERROR_RETRY, @@ -173,13 +145,12 @@ gboolean connman_agent_report_error(ConnmanAgent *agent, return FALSE; } -gboolean connman_agent_request_input(ConnmanAgent *agent, - const char *path, GHashTable *fields, - DBusGMethodInvocation *context) +static gboolean connman_agent_request_input(ConnmanAgent *agent, + const char *path, GHashTable *fields, + DBusGMethodInvocation *context) { ConnmanAgentPrivate *priv = CONNMAN_AGENT_GET_PRIVATE(agent); const char *sender = dbus_g_method_get_sender(context); - char *name = NULL, *type = NULL; char **id = NULL; PendingRequest *pendingrequest = NULL; @@ -202,8 +173,8 @@ gboolean connman_agent_request_input(ConnmanAgent *agent, return FALSE; } -gboolean connman_agent_cancel(ConnmanAgent *agent, - DBusGMethodInvocation *context) +static gboolean connman_agent_cancel(ConnmanAgent *agent, + DBusGMethodInvocation *context) { ConnmanAgentPrivate *priv = CONNMAN_AGENT_GET_PRIVATE(agent); const char *sender = dbus_g_method_get_sender(context); @@ -220,8 +191,8 @@ gboolean connman_agent_cancel(ConnmanAgent *agent, return result; } -gboolean connman_agent_release(ConnmanAgent *agent, - DBusGMethodInvocation *context) +static gboolean connman_agent_release(ConnmanAgent *agent, + DBusGMethodInvocation *context) { ConnmanAgentPrivate *priv = CONNMAN_AGENT_GET_PRIVATE(agent); const char *sender = dbus_g_method_get_sender(context); @@ -323,10 +294,8 @@ gboolean connman_agent_setup(ConnmanAgent *agent, const char *path) gboolean connman_agent_register(ConnmanAgent *agent) { ConnmanAgentPrivate *priv = CONNMAN_AGENT_GET_PRIVATE(agent); - DBusGProxy *proxy; GObject *object; GError *error = NULL; - gchar *path; debug(agent, "register agent %p", agent); @@ -407,20 +376,3 @@ void connman_agent_set_cancel_func(ConnmanAgent *agent, priv->cancel_func = func; priv->cancel_data = data; } - -void connman_agent_set_release_func(ConnmanAgent *agent, - ConnmanAgentReleaseFunc func, gpointer data) -{ - ConnmanAgentPrivate *priv = CONNMAN_AGENT_GET_PRIVATE(agent); - - priv->release_func = func; - priv->release_data = data; -} - -void connman_agent_set_debug_func(ConnmanAgent *agent, ConnmanAgentDebugFunc func, gpointer data) -{ - ConnmanAgentPrivate *priv = CONNMAN_AGENT_GET_PRIVATE(agent); - - priv->debug_func = func; - priv->debug_data = data; -} diff --git a/common/connman-client.c b/common/connman-client.c index 94c9df9..c55e25c 100644 --- a/common/connman-client.c +++ b/common/connman-client.c @@ -252,7 +252,7 @@ GtkTreeModel *connman_client_get_device_model(ConnmanClient *client) return model; } -void hash_table_value_string_insert( GHashTable *hash, gpointer key, const char *str ) +static void hash_table_value_string_insert( GHashTable *hash, gpointer key, const char *str ) { GValue *itemvalue; @@ -301,6 +301,7 @@ void connman_client_set_powered(ConnmanClient *client, const gchar *device, { ConnmanClientPrivate *priv = CONNMAN_CLIENT_GET_PRIVATE(client); DBusGProxy *proxy; + GError *error; GValue value = { 0 }; DBG("client %p device %s", client, device); @@ -315,8 +316,8 @@ void connman_client_set_powered(ConnmanClient *client, const gchar *device, g_value_init(&value, G_TYPE_BOOLEAN); g_value_set_boolean(&value, powered); - GError *error = NULL; - gboolean ret = connman_set_property(proxy, "Powered", &value, &error); + error = NULL; + connman_set_property(proxy, "Powered", &value, &error); if( error ) fprintf (stderr, "error: %s\n", error->message); diff --git a/common/connman-dbus.c b/common/connman-dbus.c index c4f6ae4..33ac623 100644 --- a/common/connman-dbus.c +++ b/common/connman-dbus.c @@ -535,7 +535,7 @@ static void manager_properties(DBusGProxy *proxy, GHashTable *hash, static void manager_services(DBusGProxy *proxy, GPtrArray *array, GError *error, gpointer user_data) { - int i; + unsigned int i; DBG("proxy %p array %p", proxy, array); @@ -544,6 +544,7 @@ static void manager_services(DBusGProxy *proxy, GPtrArray *array, for (i = 0; i < array->len; i++) { + GHashTable *props; GValueArray *item = g_ptr_array_index(array, i); DBusGObjectPath *path = (DBusGObjectPath *)g_value_get_boxed(g_value_array_get_nth(item, 0)); @@ -553,7 +554,7 @@ static void manager_services(DBusGProxy *proxy, GPtrArray *array, if (service_proxy == NULL) continue; - GHashTable *props = (GHashTable *)g_value_get_boxed(g_value_array_get_nth(item, 1)); + props = (GHashTable *)g_value_get_boxed(g_value_array_get_nth(item, 1)); service_properties(service_proxy, props, user_data); } } @@ -561,7 +562,7 @@ static void manager_services(DBusGProxy *proxy, GPtrArray *array, static void manager_technologies(DBusGProxy *proxy, GPtrArray *array, GError *error, gpointer user_data) { - int i; + unsigned int i; DBG("proxy %p array %p", proxy, array); @@ -592,7 +593,7 @@ static void services_removed(DBusGProxy *proxy, GPtrArray *array, { GtkTreeStore *store = user_data; GtkTreeIter iter; - int i; + unsigned int i; DBG("store %p proxy %p array %p", store, proxy, array); |