[Bug 755505] ges: trivial leakages

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=755505

--- Comment #10 from Justin J. Kim <justin.joy.9to5 at gmail.com> ---
Review of attachment 311999:
 --> (https://bugzilla.gnome.org/review?bug=755505&attachment=311999)

::: ges/ges-timeline-element.c
@@ -263,1 +263,3 @@
   G_OBJECT_CLASS (ges_timeline_element_parent_class)->finalize (self);
+
+  if (object_name_counts)

I thought the lifecycle of object_name_counts variable should be same with
timeline-lement because it is defined as static and local variable.

If the purpose of this quark container is to trace object number, how about
replacing this with atomic operations like 'g_atomic_int_inc' to make code
simple?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.