summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2014-12-17 11:30:43 +0100
committerThomas Haller <thaller@redhat.com>2014-12-17 12:42:28 +0100
commit3b6244c77ef3411731262d38725e2bf95cf964de (patch)
tree77f81e76ad711c088c3f7832115c7b8fb7f25f3b
parent3d5084e5fe8c52973c882b9ee70cdcc33d56dc96 (diff)
downloadnetwork-manager-applet-3b6244c77ef3411731262d38725e2bf95cf964de.tar.gz
fixup! return icon name from get_icon() whenever possible
-rw-r--r--src/applet-device-broadband.c22
-rw-r--r--src/applet.c8
2 files changed, 15 insertions, 15 deletions
diff --git a/src/applet-device-broadband.c b/src/applet-device-broadband.c
index 01d4c52c..26d77073 100644
--- a/src/applet-device-broadband.c
+++ b/src/applet-device-broadband.c
@@ -635,17 +635,17 @@ get_icon (NMDevice *device,
info = g_object_get_data (G_OBJECT (device), "devinfo");
g_assert (info);
- return mobile_helper_get_icon (device,
- state,
- connection,
- out_pixbuf,
- out_icon_name,
- tip,
- applet,
- broadband_state_to_mb_state (info),
- broadband_act_to_mb_act (info),
- mm_modem_get_signal_quality (info->mm_modem, NULL),
- (mm_modem_get_state (info->mm_modem) >= MM_MODEM_STATE_ENABLED));
+ mobile_helper_get_icon (device,
+ state,
+ connection,
+ out_pixbuf,
+ out_icon_name,
+ tip,
+ applet,
+ broadband_state_to_mb_state (info),
+ broadband_act_to_mb_act (info),
+ mm_modem_get_signal_quality (info->mm_modem, NULL),
+ (mm_modem_get_state (info->mm_modem) >= MM_MODEM_STATE_ENABLED));
}
/********************************************************************/
diff --git a/src/applet.c b/src/applet.c
index 12d3e461..da1272d6 100644
--- a/src/applet.c
+++ b/src/applet.c
@@ -2883,9 +2883,9 @@ mm1_client_setup (NMApplet *applet)
static void
applet_common_get_device_icon (NMDeviceState state,
- GdkPixbuf **out_pixbuf,
- char **out_icon_name,
- NMApplet *applet)
+ GdkPixbuf **out_pixbuf,
+ char **out_icon_name,
+ NMApplet *applet)
{
int stage = -1;
@@ -2996,7 +2996,7 @@ applet_get_device_icon_for_state (NMApplet *applet,
if (!*out_pixbuf && icon_name)
*out_pixbuf = g_object_ref (nma_icon_check_and_load (icon_name, applet));
- *out_icon_name = icon_name ? g_strdup (icon_name) : NULL;
+ *out_icon_name = g_strdup (icon_name);
if (!*out_tip)
*out_tip = get_tip_for_device_state (device, state, connection);
if (icon_name || *out_pixbuf)