Attachment 105478 Details for Bug 82925 – linux: Respect the CriticalPowerAction config option

[patch] linux: Respect the CriticalPowerAction config option

linux-Respect-the-CriticalPowerAction-config-optio.patch (text/plain), 1.30 KB, created by Bastien Nocera on 2014-08-30 17:30:45 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bastien Nocera

Created: 2014-08-30 17:30:45 UTC

Size: 1.30 KB

patch

obsolete

>From d966baa82e400128b0352b75d7e27d7a02feef09 Mon Sep 17 00:00:00 2001
>From: Bastien Nocera <hadess@hadess.net>
>Date: Mon, 25 Aug 2014 17:04:55 +0200
>Subject: [PATCH] linux: Respect the CriticalPowerAction config option
>
>It was documented, but the configuration was never actually read.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=82925
>---
> src/linux/up-backend.c | 16 ++++++++++++++--
> 1 file changed, 14 insertions(+), 2 deletions(-)
>
>diff --git a/src/linux/up-backend.c b/src/linux/up-backend.c
>index 9f4d033..f588e74 100644
>--- a/src/linux/up-backend.c
>+++ b/src/linux/up-backend.c
>@@ -371,11 +371,23 @@ up_backend_get_critical_action (UpBackend *backend)
> 		{ "Hibernate", "CanHibernate" },
> 		{ "PowerOff", NULL },
> 	};
>-	guint i;
>+	guint i = 0;
>+	char *action;
> 
> 	g_return_val_if_fail (backend->priv->logind_proxy != NULL, NULL);
> 
>-	for (i = 0; i < G_N_ELEMENTS (actions); i++) {
>+	/* Find the configured action first */
>+	action = up_config_get_string (backend->priv->config, "CriticalPowerAction");
>+	if (action != NULL) {
>+		for (i = 0; i < G_N_ELEMENTS (actions); i++)
>+			if (g_str_equal (actions[i].method, action))
>+				break;
>+		if (i >= G_N_ELEMENTS (actions))
>+			i = 0;
>+		g_free (action);
>+	}
>+
>+	for (; i < G_N_ELEMENTS (actions); i++) {
> 		GVariant *result;
> 
> 		if (actions[i].can_method) {
>-- 
>2.1.0

Actions: View | Splinter Review

Attachments on bug 82925: 105065 | 105066 | 105242 | 105243 | 105466 | 105467 | 105476 | 105477 | 105478 | 105631

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.