Revert "flacparse: push tags in pre_push_frame"
authorSebastian Dröge <sebastian@centricular.com>
Sun, 13 Mar 2016 08:33:13 +0000 (10:33 +0200)
committerSebastian Dröge <sebastian@centricular.com>
Sun, 13 Mar 2016 08:33:13 +0000 (10:33 +0200)
This reverts commit 4065fcb80a49924b70f0c8fc159dec0ff47943a1.

flacparse should not push tags by itself, the base class is going to do that
while properly merging in upstream tags. It just didn't because of a bug in
the base class, which was hidden by this commit.

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

gst/audioparsers/gstflacparse.c

index d523c2dcfc6b84f0d3b56ab8ff38d012dc021f6b..34e1e5e664a76aa675ca9462fe517ea2a610cbba 100644 (file)
@@ -1714,12 +1714,6 @@ gst_flac_parse_pre_push_frame (GstBaseParse * parse, GstBaseParseFrame * frame)
     flacparse->sent_codec_tag = TRUE;
   }
 
-  /* Push tags */
-  if (flacparse->tags) {
-    gst_pad_push_event (GST_BASE_PARSE_SRC_PAD (flacparse),
-        gst_event_new_tag (flacparse->tags));
-    flacparse->tags = NULL;
-  }
   /* Push toc */
   if (flacparse->toc) {
     gst_pad_push_event (GST_BASE_PARSE_SRC_PAD (flacparse),