gegl r3024 - in trunk: . gegl/operation



Author: martinn
Date: Sat Apr 11 07:50:13 2009
New Revision: 3024
URL: http://svn.gnome.org/viewvc/gegl?rev=3024&view=rev

Log:
gegl: Simplify gegl_operation_context_take_object()

Use g_value_take_object() instead of g_value_set_object() so we can
remove explicit unrefs in gegl_operation_context_take_object().

Modified:
   trunk/ChangeLog
   trunk/gegl/operation/gegl-operation-context.c

Modified: trunk/gegl/operation/gegl-operation-context.c
==============================================================================
--- trunk/gegl/operation/gegl-operation-context.c	(original)
+++ trunk/gegl/operation/gegl-operation-context.c	Sat Apr 11 07:50:13 2009
@@ -255,7 +255,7 @@
       GValue value = {0, };
       g_value_init (&value, G_PARAM_SPEC_VALUE_TYPE (pspec));
 
-      g_value_set_object (&value, data);
+      g_value_take_object (&value, data);
       gegl_operation_context_set_property (context, padname, &value);
 
       g_value_unset (&value);
@@ -266,11 +266,7 @@
                  G_STRFUNC,
                  data ? "we have data" : "",
                  padname);
-      if (data)
-        g_object_unref (data); /* are we stealing the initial reference? */
-      return;
     }
-  g_object_unref (data); /* are we stealing the initial reference? */
 }
 
 GObject *



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]