Blob Blame History Raw
From b770f7675a42a0ff0bc0845aff1af1a739525e3c Mon Sep 17 00:00:00 2001
From: Hans de Goede <hdegoede@redhat.com>
Date: Mon, 10 Jun 2013 10:39:16 +0200
Subject: [PATCH 05/35] cheese-camera: 2 minor error handling cleanups

1) Simplify error cleanup in cheese_camera_set_camera_source()
2) Don't call g_error_free on a possible NULL error in
   cheese_camera_element_from_effect()

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 libcheese/cheese-camera.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/libcheese/cheese-camera.c b/libcheese/cheese-camera.c
index b353152..b55dc82 100644
--- a/libcheese/cheese-camera.c
+++ b/libcheese/cheese-camera.c
@@ -426,11 +426,7 @@ cheese_camera_set_camera_source (CheeseCamera *camera)
 
   if (priv->video_source == NULL)
   {
-    if (err != NULL)
-    {
-      g_error_free (err);
-      err = NULL;
-    }
+    g_clear_error(&err);
     return FALSE;
   }
 
@@ -867,7 +863,7 @@ cheese_camera_element_from_effect (CheeseCamera *camera, CheeseEffect *effect)
   g_free (effects_pipeline_desc);
   if (!effect_filter || (err != NULL))
   {
-    g_error_free (err);
+    g_clear_error (&err);
     g_warning ("Error with effect filter %s. Ignored", name);
     g_free (name);
     return NULL;
-- 
1.8.2.1