diff options
author | Bastien Nocera <hadess@hadess.net> | 2013-10-17 00:57:27 +0200 |
---|---|---|
committer | Bastien Nocera <hadess@hadess.net> | 2013-10-17 00:57:27 +0200 |
commit | b2f72dd69ac683e7fa213d83b410b38fe1ab1b21 (patch) | |
tree | c48e1c3defefd913365d43a71811347603680f16 | |
parent | c00eda8ca0c2d8dc772b8cc8b4bdc93ab353dc29 (diff) | |
download | upower-b2f72dd69ac683e7fa213d83b410b38fe1ab1b21.tar.gz |
all: Remove WarningLevel from the daemon properties
The WarningLevel property just replicated the warning level
on the display device, or at least should have. So we fix the latter
to remove the former.
-rw-r--r-- | libupower-glib/up-client.c | 36 | ||||
-rw-r--r-- | libupower-glib/up-client.h | 1 | ||||
-rw-r--r-- | src/org.freedesktop.UPower.xml | 7 | ||||
-rw-r--r-- | src/up-daemon.c | 16 | ||||
-rw-r--r-- | tools/up-tool.c | 2 |
5 files changed, 1 insertions, 61 deletions
diff --git a/libupower-glib/up-client.c b/libupower-glib/up-client.c index ec9434d..2169a59 100644 --- a/libupower-glib/up-client.c +++ b/libupower-glib/up-client.c @@ -69,7 +69,6 @@ enum { PROP_0, PROP_DAEMON_VERSION, PROP_ON_BATTERY, - PROP_WARNING_LEVEL, PROP_LID_IS_CLOSED, PROP_LID_IS_PRESENT, PROP_IS_DOCKED, @@ -230,23 +229,6 @@ up_client_get_on_battery (UpClient *client) return up_client_glue_get_on_battery (client->priv->proxy); } -/** - * up_client_get_warning_level: - * @client: a #UpClient instance. - * - * Get whether the system is low on battery, or critically low. - * - * Return value: The #UpDeviceLevel of the whole system. - * - * Since: 1.0 - **/ -UpDeviceLevel -up_client_get_warning_level (UpClient *client) -{ - g_return_val_if_fail (UP_IS_CLIENT (client), FALSE); - return up_client_glue_get_warning_level (client->priv->proxy); -} - /* * up_client_add: */ @@ -351,9 +333,6 @@ up_client_get_property (GObject *object, case PROP_ON_BATTERY: g_value_set_boolean (value, up_client_glue_get_on_battery (client->priv->proxy)); break; - case PROP_WARNING_LEVEL: - g_value_set_uint (value, up_client_glue_get_warning_level (client->priv->proxy)); - break; case PROP_LID_IS_CLOSED: g_value_set_boolean (value, up_client_glue_get_lid_is_closed (client->priv->proxy)); break; @@ -410,21 +389,6 @@ up_client_class_init (UpClientClass *klass) FALSE, G_PARAM_READABLE)); /** - * UpClient:warning-level: - * - * The warning level e.g. %UP_DEVICE_LEVEL_WARNING. - * - * Since: 1.0 - **/ - g_object_class_install_property (object_class, - PROP_WARNING_LEVEL, - g_param_spec_uint ("warning-level", - NULL, NULL, - UP_DEVICE_LEVEL_UNKNOWN, - UP_DEVICE_LEVEL_LAST, - UP_DEVICE_LEVEL_UNKNOWN, - G_PARAM_READABLE)); - /** * UpClient:lid-is-closed: * * If the laptop lid is closed. diff --git a/libupower-glib/up-client.h b/libupower-glib/up-client.h index 0c58a3d..0f502d9 100644 --- a/libupower-glib/up-client.h +++ b/libupower-glib/up-client.h @@ -89,7 +89,6 @@ gboolean up_client_get_lid_is_closed (UpClient *client); gboolean up_client_get_lid_is_present (UpClient *client); gboolean up_client_get_is_docked (UpClient *client); gboolean up_client_get_on_battery (UpClient *client); -UpDeviceLevel up_client_get_warning_level (UpClient *client); G_END_DECLS diff --git a/src/org.freedesktop.UPower.xml b/src/org.freedesktop.UPower.xml index 2761653..d02ee3a 100644 --- a/src/org.freedesktop.UPower.xml +++ b/src/org.freedesktop.UPower.xml @@ -180,13 +180,6 @@ method return sender=:1.386 -> dest=:1.451 reply_serial=2 </doc:para></doc:description></doc:doc> </property> - <property name="WarningLevel" type="u" access="read"> - <doc:doc><doc:description><doc:para> - Indicates whether the system is running on battery power and if the batteries are low, - or critically low. This property is provided for convenience. - </doc:para></doc:description></doc:doc> - </property> - <property name="LidIsClosed" type="b" access="read"> <doc:doc> <doc:description> diff --git a/src/up-daemon.c b/src/up-daemon.c index c81e93c..22e9116 100644 --- a/src/up-daemon.c +++ b/src/up-daemon.c @@ -50,7 +50,6 @@ enum PROP_LID_IS_CLOSED, PROP_LID_IS_PRESENT, PROP_IS_DOCKED, - PROP_WARNING_LEVEL, PROP_LAST }; @@ -758,9 +757,8 @@ up_daemon_set_warning_level (UpDaemon *daemon, UpDeviceLevel warning_level) g_debug ("warning_level = %s", up_device_level_to_string (warning_level)); priv->warning_level = warning_level; - g_object_notify (G_OBJECT (daemon), "warning-level"); - up_daemon_queue_changed_property (daemon, "WarningLevel", g_variant_new_uint32 (warning_level)); + g_object_set (G_OBJECT (daemon->priv->display_device), "warning-level", warning_level, NULL); if (daemon->priv->warning_level == UP_DEVICE_LEVEL_ACTION) { if (daemon->priv->action_timeout_id == 0) { @@ -1121,9 +1119,6 @@ up_daemon_get_property (GObject *object, guint prop_id, GValue *value, GParamSpe case PROP_IS_DOCKED: g_value_set_boolean (value, priv->is_docked); break; - case PROP_WARNING_LEVEL: - g_value_set_uint (value, priv->warning_level); - break; default: G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec); break; @@ -1207,15 +1202,6 @@ up_daemon_class_init (UpDaemonClass *klass) G_PARAM_READABLE)); g_object_class_install_property (object_class, - PROP_WARNING_LEVEL, - g_param_spec_uint ("warning-level", - NULL, NULL, - UP_DEVICE_LEVEL_UNKNOWN, - UP_DEVICE_LEVEL_LAST, - UP_DEVICE_LEVEL_UNKNOWN, - G_PARAM_READABLE)); - - g_object_class_install_property (object_class, PROP_LID_IS_CLOSED, g_param_spec_boolean ("lid-is-closed", "Laptop lid is closed", diff --git a/tools/up-tool.c b/tools/up-tool.c index c77d627..99dba78 100644 --- a/tools/up-tool.c +++ b/tools/up-tool.c @@ -128,7 +128,6 @@ up_client_print (UpClient *client) g_object_get (client, "daemon-version", &daemon_version, "on-battery", &on_battery, - "warning-level", &warning_level, "lid-is-closed", &lid_is_closed, "lid-is-present", &lid_is_present, "is-docked", &is_docked, @@ -136,7 +135,6 @@ up_client_print (UpClient *client) g_print (" daemon-version: %s\n", daemon_version); g_print (" on-battery: %s\n", on_battery ? "yes" : "no"); - g_print (" warning-level: %s\n", up_device_level_to_string (warning_level)); g_print (" lid-is-closed: %s\n", lid_is_closed ? "yes" : "no"); g_print (" lid-is-present: %s\n", lid_is_present ? "yes" : "no"); g_print (" is-docked: %s\n", is_docked ? "yes" : "no"); |