From a09510f6cb416fed7e98ce7df61d74f56d7a1bff Mon Sep 17 00:00:00 2001 From: Ted Gould Date: Sat, 11 Apr 2009 23:07:38 -0500 Subject: Fixing some pedantic little bugs to try and figure out what's causing 359018 --- libindicate/indicator.c | 8 ++++---- libindicate/listener.c | 4 ++-- libindicate/server.c | 9 ++++++--- 3 files changed, 12 insertions(+), 9 deletions(-) diff --git a/libindicate/indicator.c b/libindicate/indicator.c index 9427d03..7e0c986 100644 --- a/libindicate/indicator.c +++ b/libindicate/indicator.c @@ -261,7 +261,7 @@ indicate_indicator_set_property_icon (IndicateIndicator * indicator, const gchar return; } - gchar * prop_str = g_base64_encode(png_data, png_data_len); + gchar * prop_str = g_base64_encode((guchar *)png_data, png_data_len); indicate_indicator_set_property(indicator, key, prop_str); g_free(prop_str); @@ -308,7 +308,7 @@ set_property (IndicateIndicator * indicator, const gchar * key, const gchar * da { g_return_if_fail(INDICATE_IS_INDICATOR(indicator)); - if (key != NULL && !strcmp(key, "type")) { + if (key != NULL && !g_strcmp0(key, "type")) { g_warning("Trying to set the 'type' of an indicator which should be done through subclassing."); return; } @@ -316,7 +316,7 @@ set_property (IndicateIndicator * indicator, const gchar * key, const gchar * da IndicateIndicatorPrivate * priv = INDICATE_INDICATOR_GET_PRIVATE(indicator); gchar * current = g_hash_table_lookup(priv->properties, key); - if (current == NULL || strcmp(current, data)) { + if (current == NULL || g_strcmp0(current, data)) { /* If the value has changed or there is no value */ gchar * newkey = g_strdup(key); /* g_debug("What is newkey? %s", newkey); */ @@ -335,7 +335,7 @@ get_property (IndicateIndicator * indicator, const gchar * key) { g_return_val_if_fail(INDICATE_IS_INDICATOR(indicator), NULL); - if (key != NULL && !strcmp(key, "type")) { + if (key != NULL && !g_strcmp0(key, "type")) { return indicate_indicator_get_indicator_type(indicator); } diff --git a/libindicate/listener.c b/libindicate/listener.c index 381a5f9..4c754ac 100644 --- a/libindicate/listener.c +++ b/libindicate/listener.c @@ -272,7 +272,7 @@ indicate_listener_init (IndicateListener * listener) static void indicate_listener_finalize (GObject * obj) { - IndicateListener * listener = INDICATE_LISTENER(obj); + /* IndicateListener * listener = INDICATE_LISTENER(obj); */ return; } @@ -833,7 +833,7 @@ gboolean _indicate_listener_get_indicator_servers (IndicateListener * listener, GList * servers) { - + return TRUE; } static void diff --git a/libindicate/server.c b/libindicate/server.c index 9e33516..f2ebdd2 100644 --- a/libindicate/server.c +++ b/libindicate/server.c @@ -307,6 +307,9 @@ set_property (GObject * obj, guint id, const GValue * value, GParamSpec * pspec) g_return_if_fail(id == PROP_DESKTOP || id == PROP_TYPE); gchar ** outstr; + gchar * tempstr = NULL; + outstr = &tempstr; + IndicateServerPrivate * priv = INDICATE_SERVER_GET_PRIVATE(obj); switch (id) { case PROP_DESKTOP: @@ -331,7 +334,7 @@ get_property (GObject * obj, guint id, GValue * value, GParamSpec * pspec) { g_return_if_fail(id == PROP_DESKTOP || id == PROP_TYPE); - gchar * outstr; + gchar * outstr = NULL; IndicateServerPrivate * priv = INDICATE_SERVER_GET_PRIVATE(obj); switch (id) { case PROP_DESKTOP: @@ -743,7 +746,7 @@ count_by_type (IndicateIndicator * indicator, count_by_t * cbt) if (type == NULL && cbt->type == NULL) { cbt->count++; } else if (type == NULL || cbt->type == NULL) { - } else if (!strcmp(type, cbt->type)) { + } else if (!g_strcmp0(type, cbt->type)) { cbt->count++; } @@ -826,7 +829,7 @@ get_indicator_list_by_type (IndicateServer * server, gchar * type, GArray ** ind if (type == NULL && itype == NULL) { g_array_insert_val(*indicators, i++, id); } else if (type == NULL || itype == NULL) { - } else if (!strcmp(type, itype)) { + } else if (!g_strcmp0(type, itype)) { g_array_insert_val(*indicators, i++, id); } } -- cgit v1.2.3