[Goodies-commits] r7370 - in xfce4-power-manager/trunk: . src

Ali Abdallah aliov at xfce.org
Tue May 19 12:28:02 CEST 2009


Author: aliov
Date: 2009-05-19 10:28:02 +0000 (Tue, 19 May 2009)
New Revision: 7370

Modified:
   xfce4-power-manager/trunk/ChangeLog
   xfce4-power-manager/trunk/src/xfpm-brightness-hal.c
Log:
	* : Use GError in the get brightness method.

Modified: xfce4-power-manager/trunk/ChangeLog
===================================================================
--- xfce4-power-manager/trunk/ChangeLog	2009-05-19 10:04:51 UTC (rev 7369)
+++ xfce4-power-manager/trunk/ChangeLog	2009-05-19 10:28:02 UTC (rev 7370)
@@ -1,3 +1,6 @@
+2009-05-19 12:30
+	* : Use GError in the get brightness method.
+
 2009-05-19 12:00
 	* : check brightness value and only display relevent values in
 	the brightness popup.

Modified: xfce4-power-manager/trunk/src/xfpm-brightness-hal.c
===================================================================
--- xfce4-power-manager/trunk/src/xfpm-brightness-hal.c	2009-05-19 10:04:51 UTC (rev 7369)
+++ xfce4-power-manager/trunk/src/xfpm-brightness-hal.c	2009-05-19 10:28:02 UTC (rev 7370)
@@ -86,23 +86,16 @@
 G_DEFINE_TYPE(XfpmBrightnessHal, xfpm_brightness_hal, G_TYPE_OBJECT)
 
 static gint 
-xfpm_brightness_hal_get_level (XfpmBrightnessHal *brg)
+xfpm_brightness_hal_get_level (XfpmBrightnessHal *brg, GError **error)
 {
-    GError *error = NULL;
     gint level = 0;
     gboolean ret = FALSE;
     
-    ret = dbus_g_proxy_call (brg->priv->proxy, "GetBrightness", &error,
+    ret = dbus_g_proxy_call (brg->priv->proxy, "GetBrightness", error,
 	 		     G_TYPE_INVALID,
 			     G_TYPE_INT, &level,
 			     G_TYPE_INVALID);
 
-    if ( error )
-    {
-	g_critical ("Error getting brightness level: %s\n", error->message);
-	g_error_free (error);
-    }
-		       
     if (!ret)
     {
 	g_warning("GetBrightness failed\n");
@@ -195,7 +188,7 @@
     xfpm_brightness_hal_get_device (brg, udi[0]);
     xfpm_brightness_hal_set_proxy (brg, udi[0]);
 
-    brg->priv->hw_level = xfpm_brightness_hal_get_level (brg);
+    brg->priv->hw_level = xfpm_brightness_hal_get_level (brg, NULL);
     TRACE ("Current hw level =%d\n", brg->priv->hw_level);
     
     hal_manager_free_string_array (udi);
@@ -205,6 +198,8 @@
 static void
 xfpm_brightness_hal_up (XfpmBrightnessHal *brg)
 {
+    GError *error = NULL;
+    
     gboolean enable_brightness = xfpm_xfconf_get_property_bool (brg->priv->conf, ENABLE_BRIGHTNESS_CONTROL);
     
     if ( enable_brightness == FALSE || brg->priv->brightness_in_hw)
@@ -217,14 +212,23 @@
     }
     
 signal:
-    brg->priv->hw_level = xfpm_brightness_hal_get_level (brg);
-    if ( G_LIKELY ( brg->priv->hw_level != 0 ) )
-	g_signal_emit (G_OBJECT (brg), signals [BRIGHTNESS_UP], 0, brg->priv->hw_level);
+    brg->priv->hw_level = xfpm_brightness_hal_get_level (brg, &error);
+   
+    if ( error )
+    {
+	g_warning ("Error getting brightness level: %s\n", error->message);
+	g_error_free (error);
+	return;
+    }
+	
+    g_signal_emit (G_OBJECT (brg), signals [BRIGHTNESS_UP], 0, brg->priv->hw_level);
 }
 
 static void
 xfpm_brightness_hal_down (XfpmBrightnessHal *brg)
 {
+    GError *error = NULL;
+    
     gboolean enable_brightness = xfpm_xfconf_get_property_bool (brg->priv->conf, ENABLE_BRIGHTNESS_CONTROL);
     
     if ( enable_brightness == FALSE || brg->priv->brightness_in_hw)
@@ -237,9 +241,15 @@
     }
     
 signal:
-    brg->priv->hw_level = xfpm_brightness_hal_get_level (brg);
-    if ( G_LIKELY ( brg->priv->hw_level != 0 ) )
-	g_signal_emit (G_OBJECT (brg), signals [BRIGHTNESS_DOWN], 0, brg->priv->hw_level);
+    brg->priv->hw_level = xfpm_brightness_hal_get_level (brg, &error);
+    
+    if ( error )
+    {
+	g_warning ("Error getting brightness level: %s\n", error->message);
+	g_error_free (error);
+	return;
+    }
+    g_signal_emit (G_OBJECT (brg), signals [BRIGHTNESS_UP], 0, brg->priv->hw_level);
 }
 
 static void
@@ -260,6 +270,7 @@
 static void
 xfpm_brightness_hal_reset_cb (XfpmIdle *idle, XfpmBrightnessHal *brg)
 {
+    GError *error = NULL;
     guint level;
     
     if (brg->priv->block)
@@ -268,8 +279,15 @@
     if ( brg->priv->inhibited )
 	return;
     
-    level = xfpm_brightness_hal_get_level(brg);
+    level = xfpm_brightness_hal_get_level (brg, &error);
      
+    if ( error )
+    {
+	g_warning ("Error getting brightness level: %s\n", error->message);
+	g_error_free (error);
+	return;
+    }
+     
     if ( level != brg->priv->hw_level )
     {
 	TRACE("Resetting brightness level to %d", brg->priv->hw_level);
@@ -280,13 +298,21 @@
 static void
 xfpm_brightness_timeout_on_ac (XfpmBrightnessHal *brg)
 {
+    GError *error = NULL;
     guint level;
     
     if ( brg->priv->on_battery )
 	    return;
     
-    level = xfpm_brightness_hal_get_level(brg);
+    level = xfpm_brightness_hal_get_level (brg, &error);
     
+    if ( error )
+    {
+	g_warning ("Error getting brightness level: %s\n", error->message);
+	g_error_free (error);
+	return;
+    }
+    
     if ( level != 0 && level != brg->priv->step )
     {
 	TRACE ("Reducing brightness, on AC power\n");
@@ -297,13 +323,21 @@
 static void
 xfpm_brightness_timeout_on_battery (XfpmBrightnessHal *brg)
 {
+    GError *error = NULL;
     guint level;
     
     if ( !brg->priv->on_battery )
 	    return;
     
-    level = xfpm_brightness_hal_get_level(brg);
+    level = xfpm_brightness_hal_get_level (brg, &error);
     
+    if ( error )
+    {
+	g_warning ("Error getting brightness level: %s\n", error->message);
+	g_error_free (error);
+	return;
+    }
+    
     if ( level != 0 && level != brg->priv->step )
     {
 	xfpm_brightness_hal_set_level(brg, brg->priv->step);




More information about the Goodies-commits mailing list