[gstreamer-bugs] [Bug 395554] gst_tag_setter_merge_tags works at most once

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=395554

  GStreamer | gstreamer (core) | Ver: HEAD CVS


Tim-Philipp Müller changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #80066|none                        |committed
               Flag|                            |
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |0.10.12




------- Comment #2 from Tim-Philipp Müller  2007-01-12 10:49 UTC -------

 2007-01-11  Tim-Philipp Müller  <tim at centricular dot net>

        Patch by: René Stadler <mail at renestadler dot de>

        * gst/gsttagsetter.c: (gst_tag_setter_merge_tags):
          gst_tag_list_merge() returns a new list, so it's not the best idea
          to ingore its return value. Effectively meant that tags could only
          be merged on a GstTagSetter once using _merge_tags(). Fixes #395554.
          Also add function guard to require a non-NULL taglist as input (has
          always been so due to gst_tag_list_copy(), just making it explicit).


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email