[Bug 796452] avdemux:fix memory leak

[GStreamer (GNOME Bugzilla)]

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

Edward Hervey <bilboed at bilboed.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #372486|none                        |needs-work
             status|                            |

--- Comment #5 from Edward Hervey <bilboed at bilboed.com> ---
Review of attachment 372486:
 --> (https://bugzilla.gnome.org/review?bug=796452&attachment=372486)

No I meant something like the following. That table can just be statically
present.

static struct {
 const gchar *ffmpeg_tag_name;
 const gchar *gst_tag_name;
} tagmapping[] = {
 { "album", GST_TAG_ALBUM },
 ...
};

...

for (i = 0; i < G_N_ELEMENTS(tagmapping); i++) {
  if (!g_strcmp0(tagmapping[i].ffmpeg_tag_name, something))
    /* Do something */
}

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