[gnome-control-center] power: Use g_auto for variables
- From: Georges Basile Stavracas Neto <gbsneto src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-control-center] power: Use g_auto for variables
- Date: Sun, 3 Jun 2018 01:51:37 +0000 (UTC)
commit 3622c8ff141fd02b83a6204eaa966fed736d4db5
Author: Robert Ancell <robert ancell canonical com>
Date: Wed May 30 12:03:58 2018 +1200
power: Use g_auto for variables
panels/power/cc-power-panel.c | 197 ++++++++++++++----------------------------
1 file changed, 66 insertions(+), 131 deletions(-)
---
diff --git a/panels/power/cc-power-panel.c b/panels/power/cc-power-panel.c
index a07a04f90..f958e050c 100644
--- a/panels/power/cc-power-panel.c
+++ b/panels/power/cc-power-panel.c
@@ -180,11 +180,10 @@ no_prelight_row_new (void)
static char *
get_chassis_type (GCancellable *cancellable)
{
- char *ret = NULL;
- GError *error = NULL;
- GVariant *inner;
- GVariant *variant = NULL;
- GDBusConnection *connection;
+ g_autoptr(GError) error = NULL;
+ g_autoptr(GVariant) inner = NULL;
+ g_autoptr(GVariant) variant = NULL;
+ g_autoptr(GDBusConnection) connection = NULL;
connection = g_bus_get_sync (G_BUS_TYPE_SYSTEM,
cancellable,
@@ -193,8 +192,7 @@ get_chassis_type (GCancellable *cancellable)
{
if (!g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
g_warning ("system bus not available: %s", error->message);
- g_error_free (error);
- goto out;
+ return NULL;
}
variant = g_dbus_connection_call_sync (connection,
@@ -214,18 +212,11 @@ get_chassis_type (GCancellable *cancellable)
{
if (!g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
g_debug ("Failed to get property '%s': %s", "Chassis", error->message);
- g_error_free (error);
- goto out;
+ return NULL;
}
g_variant_get (variant, "(v)", &inner);
- ret = g_variant_dup_string (inner, NULL);
- g_variant_unref (inner);
-
-out:
- g_clear_object (&connection);
- g_clear_pointer (&variant, g_variant_unref);
- return ret;
+ return g_variant_dup_string (inner, NULL);
}
static gchar *
@@ -279,7 +270,7 @@ get_details_string (gdouble percentage, UpDeviceState state, guint64 time)
if (time > 0)
{
- gchar *time_string;
+ g_autofree gchar *time_string = NULL;
time_string = get_timestring (time);
switch (state)
@@ -314,7 +305,6 @@ get_details_string (gdouble percentage, UpDeviceState state, guint64 time)
details = g_strdup_printf ("error: %s", up_device_state_to_string (state));
break;
}
- g_free (time_string);
}
else
{
@@ -351,13 +341,13 @@ get_details_string (gdouble percentage, UpDeviceState state, guint64 time)
static void
set_primary (CcPowerPanel *panel, UpDevice *device)
{
- gchar *details = NULL;
+ g_autofree gchar *details = NULL;
gdouble percentage;
guint64 time_empty, time_full, time;
UpDeviceState state;
GtkWidget *box, *box2, *label;
GtkWidget *levelbar, *row;
- gchar *s;
+ g_autofree gchar *s = NULL;
gdouble energy_full, energy_rate;
g_object_get (device,
@@ -405,7 +395,6 @@ set_primary (CcPowerPanel *panel, UpDevice *device)
s = g_strdup_printf ("%d%%", (int)(percentage + 0.5));
label = gtk_label_new (s);
- g_free (s);
gtk_widget_set_halign (label, GTK_ALIGN_END);
gtk_style_context_add_class (gtk_widget_get_style_context (label), GTK_STYLE_CLASS_DIM_LABEL);
gtk_box_pack_start (GTK_BOX (box2), label, FALSE, TRUE, 0);
@@ -420,8 +409,6 @@ set_primary (CcPowerPanel *panel, UpDevice *device)
g_object_set_data (G_OBJECT (row), "primary", GINT_TO_POINTER (TRUE));
- g_free (details);
-
gtk_widget_set_visible (panel->battery_section, TRUE);
}
@@ -437,8 +424,8 @@ add_battery (CcPowerPanel *panel, UpDevice *device)
GtkWidget *label;
GtkWidget *levelbar;
GtkWidget *widget;
- gchar *s;
- gchar *icon_name;
+ g_autofree gchar *s = NULL;
+ g_autofree gchar *icon_name = NULL;
const gchar *name;
g_object_get (device,
@@ -485,7 +472,6 @@ add_battery (CcPowerPanel *panel, UpDevice *device)
s = g_strdup_printf ("%d%%", (int)percentage);
label = gtk_label_new (s);
- g_free (s);
gtk_widget_set_halign (label, GTK_ALIGN_END);
gtk_style_context_add_class (gtk_widget_get_style_context (label), GTK_STYLE_CLASS_DIM_LABEL);
gtk_box_pack_start (GTK_BOX (box2), label, FALSE, TRUE, 0);
@@ -510,8 +496,6 @@ add_battery (CcPowerPanel *panel, UpDevice *device)
gtk_size_group_add_widget (panel->row_sizegroup, row);
gtk_widget_show_all (row);
- g_free (icon_name);
-
gtk_widget_set_visible (panel->battery_section, TRUE);
}
@@ -576,15 +560,14 @@ add_device (CcPowerPanel *panel, UpDevice *device)
GtkWidget *hbox;
GtkWidget *box2;
GtkWidget *widget;
- GString *status;
- GString *description;
+ g_autoptr(GString) status = NULL;
+ g_autoptr(GString) description = NULL;
gdouble percentage;
- gchar *name;
+ g_autofree gchar *name = NULL;
gboolean show_caution = FALSE;
gboolean is_present;
UpDeviceLevel battery_level;
- name = NULL;
g_object_get (device,
"kind", &kind,
"percentage", &percentage,
@@ -595,10 +578,7 @@ add_device (CcPowerPanel *panel, UpDevice *device)
battery_level = get_battery_level (device);
if (!is_present)
- {
- g_free (name);
- return;
- }
+ return;
if (kind == UP_DEVICE_KIND_UPS)
show_caution = TRUE;
@@ -608,8 +588,6 @@ add_device (CcPowerPanel *panel, UpDevice *device)
else
description = g_string_new (name);
- g_free (name);
-
switch (state)
{
case UP_DEVICE_STATE_CHARGING:
@@ -670,11 +648,10 @@ add_device (CcPowerPanel *panel, UpDevice *device)
if (battery_level == UP_DEVICE_LEVEL_NONE)
{
- gchar *s;
+ g_autofree gchar *s = NULL;
s = g_strdup_printf ("%d%%", (int)(percentage + 0.5));
widget = gtk_label_new (s);
- g_free (s);
}
else
{
@@ -700,9 +677,6 @@ add_device (CcPowerPanel *panel, UpDevice *device)
gtk_size_group_add_widget (panel->row_sizegroup, row);
g_object_set_data (G_OBJECT (row), "kind", GINT_TO_POINTER (kind));
- g_string_free (description, TRUE);
- g_string_free (status, TRUE);
-
gtk_widget_set_visible (panel->device_section, TRUE);
}
@@ -711,24 +685,24 @@ up_client_changed (UpClient *client,
UpDevice *device,
CcPowerPanel *self)
{
- GList *children, *l;
+ g_autoptr(GList) battery_children = NULL;
+ g_autoptr(GList) device_children = NULL;
+ GList *l;
gint i;
UpDeviceKind kind;
guint n_batteries;
gboolean on_ups;
- UpDevice *composite;
- gchar *s;
+ g_autoptr(UpDevice) composite = NULL;
+ g_autofree gchar *s = NULL;
- children = gtk_container_get_children (GTK_CONTAINER (self->battery_list));
- for (l = children; l != NULL; l = l->next)
+ battery_children = gtk_container_get_children (GTK_CONTAINER (self->battery_list));
+ for (l = battery_children; l != NULL; l = l->next)
gtk_container_remove (GTK_CONTAINER (self->battery_list), l->data);
- g_list_free (children);
gtk_widget_hide (self->battery_section);
- children = gtk_container_get_children (GTK_CONTAINER (self->device_list));
- for (l = children; l != NULL; l = l->next)
+ device_children = gtk_container_get_children (GTK_CONTAINER (self->device_list));
+ for (l = device_children; l != NULL; l = l->next)
gtk_container_remove (GTK_CONTAINER (self->device_list), l->data);
- g_list_free (children);
gtk_widget_hide (self->device_section);
#ifdef TEST_FAKE_DEVICES
@@ -813,7 +787,6 @@ up_client_changed (UpClient *client,
else
s = g_strdup_printf ("<b>%s</b>", _("Battery"));
gtk_label_set_label (GTK_LABEL (self->battery_heading), s);
- g_free (s);
if (!on_ups && n_batteries > 1)
set_primary (self, composite);
@@ -843,8 +816,6 @@ up_client_changed (UpClient *client,
add_device (self, device);
}
}
-
- g_object_unref (composite);
}
static void
@@ -886,8 +857,8 @@ static void
set_brightness_cb (GObject *source_object, GAsyncResult *res, gpointer user_data)
{
CcPowerPanel *self = CC_POWER_PANEL (user_data);
- GError *error = NULL;
- GVariant *result;
+ g_autoptr(GError) error = NULL;
+ g_autoptr(GVariant) result = NULL;
GDBusProxy *proxy = G_DBUS_PROXY (source_object);
/* not setting, so pay attention to changed signals */
@@ -901,7 +872,6 @@ set_brightness_cb (GObject *source_object, GAsyncResult *res, gpointer user_data
{
if (!g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
g_printerr ("Error setting brightness: %s\n", error->message);
- g_error_free (error);
return;
}
}
@@ -911,7 +881,7 @@ brightness_slider_value_changed_cb (GtkRange *range, gpointer user_data)
{
CcPowerPanel *self = CC_POWER_PANEL (user_data);
guint percentage;
- GVariant *variant;
+ g_autoptr(GVariant) variant = NULL;
GDBusProxy *proxy;
percentage = (guint) gtk_range_get_value (range);
@@ -946,7 +916,7 @@ brightness_slider_value_changed_cb (GtkRange *range, gpointer user_data)
/* push this to g-s-d */
g_dbus_proxy_call (proxy,
"org.freedesktop.DBus.Properties.Set",
- variant,
+ g_variant_ref_sink (variant),
G_DBUS_CALL_FLAGS_NONE,
-1,
self->cancellable,
@@ -957,7 +927,7 @@ brightness_slider_value_changed_cb (GtkRange *range, gpointer user_data)
static void
sync_kbd_brightness (CcPowerPanel *self)
{
- GVariant *result;
+ g_autoptr(GVariant) result = NULL;
gint brightness;
gboolean visible;
GtkRange *range;
@@ -984,14 +954,13 @@ sync_kbd_brightness (CcPowerPanel *self)
self->kbd_setting_brightness = TRUE;
gtk_range_set_value (range, brightness);
self->kbd_setting_brightness = FALSE;
- g_variant_unref (result);
}
}
static void
sync_screen_brightness (CcPowerPanel *self)
{
- GVariant *result;
+ g_autoptr(GVariant) result = NULL;
gint brightness;
gboolean visible;
GtkRange *range;
@@ -1020,7 +989,6 @@ sync_screen_brightness (CcPowerPanel *self)
self->setting_brightness = TRUE;
gtk_range_set_value (range, brightness);
self->setting_brightness = FALSE;
- g_variant_unref (result);
}
}
@@ -1041,26 +1009,19 @@ als_enabled_state_changed (CcPowerPanel *self)
gboolean enabled;
gboolean has_brightness = FALSE;
gboolean visible = FALSE;
- GVariant *v;
if (self->screen_proxy != NULL)
{
- v = g_dbus_proxy_get_cached_property (self->screen_proxy, "Brightness");
+ g_autoptr(GVariant) v = g_dbus_proxy_get_cached_property (self->screen_proxy, "Brightness");
if (v != NULL)
- {
- has_brightness = g_variant_get_int32 (v) >= 0.0;
- g_variant_unref (v);
- }
+ has_brightness = g_variant_get_int32 (v) >= 0.0;
}
if (self->iio_proxy != NULL)
{
- v = g_dbus_proxy_get_cached_property (self->iio_proxy, "HasAmbientLight");
+ g_autoptr(GVariant) v = g_dbus_proxy_get_cached_property (self->iio_proxy, "HasAmbientLight");
if (v != NULL)
- {
- visible = g_variant_get_boolean (v);
- g_variant_unref (v);
- }
+ visible = g_variant_get_boolean (v);
}
enabled = g_settings_get_boolean (self->gsd_settings, "ambient-enabled");
@@ -1084,7 +1045,7 @@ on_screen_property_change (GDBusProxy *proxy,
static void
got_screen_proxy_cb (GObject *source_object, GAsyncResult *res, gpointer user_data)
{
- GError *error = NULL;
+ g_autoptr(GError) error = NULL;
CcPowerPanel *self;
GDBusProxy *screen_proxy;
@@ -1093,7 +1054,6 @@ got_screen_proxy_cb (GObject *source_object, GAsyncResult *res, gpointer user_da
{
if (!g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
g_printerr ("Error creating screen proxy: %s\n", error->message);
- g_error_free (error);
return;
}
@@ -1122,7 +1082,7 @@ static void
got_kbd_proxy_cb (GObject *source_object, GAsyncResult *res, gpointer user_data)
{
CcPowerPanel *self = CC_POWER_PANEL (user_data);
- GError *error = NULL;
+ g_autoptr(GError) error = NULL;
GDBusProxy *kbd_proxy;
kbd_proxy = cc_object_storage_create_dbus_proxy_finish (res, &error);
@@ -1130,7 +1090,6 @@ got_kbd_proxy_cb (GObject *source_object, GAsyncResult *res, gpointer user_data)
{
if (!g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
g_printerr ("Error creating keyboard proxy: %s\n", error->message);
- g_error_free (error);
return;
}
@@ -1172,8 +1131,10 @@ combo_time_changed_cb (GtkWidget *widget, CcPowerPanel *self)
static char *
time_to_string_text (gint64 msecs)
{
- char *secs, *mins, *hours, *string;
int sec, min, hour, _time;
+ g_autofree gchar *hours;
+ g_autofree gchar *mins;
+ g_autofree gchar *secs;
_time = (int) (msecs / 1000);
sec = _time % 60;
@@ -1190,26 +1151,19 @@ time_to_string_text (gint64 msecs)
secs = g_strdup_printf (g_dngettext (GETTEXT_PACKAGE, "%d second",
"%d seconds", sec), sec);
- if (hour > 0)
- {
+ if (hour > 0) {
/* 5 hours 2 minutes 12 seconds */
- string = g_strdup_printf (C_("time", "%s %s %s"), hours, mins, secs);
+ return g_strdup_printf (C_("time", "%s %s %s"), hours, mins, secs);
} else if (min > 0) {
/* 2 minutes 12 seconds */
- string = g_strdup_printf (C_("time", "%s %s"), mins, secs);
+ return g_strdup_printf (C_("time", "%s %s"), mins, secs);
} else if (sec > 0) {
/* 10 seconds */
- string = g_strdup (secs);
+ return g_strdup (secs);
} else {
/* 0 seconds */
- string = g_strdup (_("0 seconds"));
+ return g_strdup (_("0 seconds"));
}
-
- g_free (hours);
- g_free (mins);
- g_free (secs);
-
- return string;
}
static void
@@ -1241,7 +1195,7 @@ set_value_for_combo (GtkComboBox *combo_box, gint value)
else if (value_tmp > value)
{
GtkTreeIter new;
- char *text;
+ g_autofree gchar *text = NULL;
/* This is an unlisted value, add it to the drop-down */
gtk_list_store_insert_before (GTK_LIST_STORE (model), &new, &iter);
@@ -1250,7 +1204,6 @@ set_value_for_combo (GtkComboBox *combo_box, gint value)
ACTION_MODEL_TEXT, text,
ACTION_MODEL_VALUE, value,
-1);
- g_free (text);
gtk_combo_box_set_active_iter (combo_box, &new);
return;
}
@@ -1340,11 +1293,11 @@ static void
bt_powered_state_changed (CcPowerPanel *panel)
{
gboolean powered, has_airplane_mode;
- GVariant *v;
+ g_autoptr(GVariant) v1 = NULL;
+ g_autoptr(GVariant) v2 = NULL;
- v = g_dbus_proxy_get_cached_property (panel->bt_rfkill, "BluetoothHasAirplaneMode");
- has_airplane_mode = g_variant_get_boolean (v);
- g_variant_unref (v);
+ v1 = g_dbus_proxy_get_cached_property (panel->bt_rfkill, "BluetoothHasAirplaneMode");
+ has_airplane_mode = g_variant_get_boolean (v1);
if (!has_airplane_mode)
{
@@ -1353,9 +1306,8 @@ bt_powered_state_changed (CcPowerPanel *panel)
return;
}
- v = g_dbus_proxy_get_cached_property (panel->bt_rfkill, "BluetoothAirplaneMode");
- powered = !g_variant_get_boolean (v);
- g_variant_unref (v);
+ v2 = g_dbus_proxy_get_cached_property (panel->bt_rfkill, "BluetoothAirplaneMode");
+ powered = !g_variant_get_boolean (v2);
g_debug ("bt powered state changed to %s", powered ? "on" : "off");
@@ -1525,7 +1477,7 @@ nm_client_ready_cb (GObject *source_object,
{
CcPowerPanel *self;
NMClient *client;
- GError *error = NULL;
+ g_autoptr(GError) error = NULL;
client = nm_client_new_finish (res, &error);
if (!client)
@@ -1539,7 +1491,6 @@ nm_client_ready_cb (GObject *source_object,
gtk_widget_set_sensitive (self->wifi_row, FALSE);
gtk_widget_set_sensitive (self->mobile_row, FALSE);
}
- g_error_free (error);
return;
}
@@ -1714,7 +1665,7 @@ iio_proxy_appeared_cb (GDBusConnection *connection,
gpointer user_data)
{
CcPowerPanel *self = CC_POWER_PANEL (user_data);
- GError *error = NULL;
+ g_autoptr(GError) error = NULL;
self->iio_proxy =
cc_object_storage_create_dbus_proxy_sync (G_BUS_TYPE_SYSTEM,
@@ -1727,7 +1678,6 @@ iio_proxy_appeared_cb (GDBusConnection *connection,
{
if (!g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
g_warning ("Could not create IIO sensor proxy: %s", error->message);
- g_error_free (error);
return;
}
@@ -1756,13 +1706,12 @@ add_power_saving_section (CcPowerPanel *self)
GtkWidget *sw;
GtkWidget *w;
int value;
- gchar *s;
+ g_autofree gchar *s = NULL;
vbox = WID (self->builder, "vbox_power");
s = g_strdup_printf ("<b>%s</b>", _("Power Saving"));
label = gtk_label_new (s);
- g_free (s);
gtk_label_set_use_markup (GTK_LABEL (label), TRUE);
gtk_widget_set_halign (label, GTK_ALIGN_START);
gtk_widget_set_margin_bottom (label, 12);
@@ -2182,10 +2131,9 @@ static gboolean
can_suspend_or_hibernate (CcPowerPanel *self,
const char *method_name)
{
- GDBusConnection *connection;
- GVariant *variant;
- gboolean result = FALSE;
- GError *error = NULL;
+ g_autoptr(GDBusConnection) connection = NULL;
+ g_autoptr(GVariant) variant = NULL;
+ g_autoptr(GError) error = NULL;
const char *s;
connection = g_bus_get_sync (G_BUS_TYPE_SYSTEM,
@@ -2195,8 +2143,7 @@ can_suspend_or_hibernate (CcPowerPanel *self,
{
if (!g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
g_warning ("system bus not available: %s", error->message);
- g_error_free (error);
- goto out;
+ return FALSE;
}
variant = g_dbus_connection_call_sync (connection,
@@ -2210,23 +2157,16 @@ can_suspend_or_hibernate (CcPowerPanel *self,
-1,
self->cancellable,
&error);
- g_object_unref (connection);
if (!variant)
{
if (!g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
g_debug ("Failed to call %s(): %s", method_name, error->message);
- g_error_free (error);
- goto out;
+ return FALSE;
}
g_variant_get (variant, "(&s)", &s);
- if (g_strcmp0 (s, "yes") == 0)
- result = TRUE;
- g_variant_unref (variant);
-
- out:
- return result;
+ return g_strcmp0 (s, "yes") == 0;
}
static void
@@ -2235,7 +2175,7 @@ add_suspend_and_power_off_section (CcPowerPanel *self)
GtkWidget *vbox;
GtkWidget *widget, *box, *label;
GtkWidget *sw, *row;
- gchar *s;
+ g_autofree gchar *s = NULL;
gint value;
GtkWidget *dialog;
GtkWidget *combo;
@@ -2272,7 +2212,6 @@ add_suspend_and_power_off_section (CcPowerPanel *self)
/* Frame header */
s = g_markup_printf_escaped ("<b>%s</b>", _("Suspend & Power Button"));
label = gtk_label_new (s);
- g_free (s);
gtk_label_set_use_markup (GTK_LABEL (label), TRUE);
gtk_widget_set_halign (label, GTK_ALIGN_START);
gtk_widget_set_margin_bottom (label, 12);
@@ -2437,7 +2376,7 @@ add_battery_section (CcPowerPanel *self)
GtkWidget *vbox;
GtkWidget *widget, *box;
GtkWidget *frame;
- gchar *s;
+ g_autofree gchar *s = NULL;
vbox = WID (self->builder, "vbox_power");
@@ -2447,7 +2386,6 @@ add_battery_section (CcPowerPanel *self)
s = g_markup_printf_escaped ("<b>%s</b>", _("Battery"));
self->battery_heading = widget = gtk_label_new (s);
- g_free (s);
gtk_label_set_use_markup (GTK_LABEL (widget), TRUE);
gtk_widget_set_halign (widget, GTK_ALIGN_START);
gtk_widget_set_margin_bottom (widget, 12);
@@ -2485,7 +2423,7 @@ add_device_section (CcPowerPanel *self)
GtkWidget *vbox;
GtkWidget *widget, *box;
GtkWidget *frame;
- gchar *s;
+ g_autofree gchar *s = NULL;
vbox = WID (self->builder, "vbox_power");
@@ -2496,7 +2434,6 @@ add_device_section (CcPowerPanel *self)
s = g_markup_printf_escaped ("<b>%s</b>", _("Devices"));
self->device_heading = widget = gtk_label_new (s);
- g_free (s);
gtk_label_set_use_markup (GTK_LABEL (widget), TRUE);
gtk_widget_set_halign (widget, GTK_ALIGN_START);
gtk_widget_set_margin_bottom (widget, 12);
@@ -2549,7 +2486,7 @@ on_content_size_changed (GtkWidget *widget, GtkAllocation *allocation, gpointer
static void
cc_power_panel_init (CcPowerPanel *self)
{
- GError *error;
+ g_autoptr(GError) error = NULL;
GtkWidget *widget;
GtkWidget *box;
guint i;
@@ -2558,7 +2495,6 @@ cc_power_panel_init (CcPowerPanel *self)
self->builder = gtk_builder_new ();
- error = NULL;
gtk_builder_add_from_resource (self->builder,
"/org/gnome/control-center/power/power.ui",
&error);
@@ -2566,7 +2502,6 @@ cc_power_panel_init (CcPowerPanel *self)
if (error != NULL)
{
g_warning ("Could not load interface file: %s", error->message);
- g_error_free (error);
return;
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]