[gstreamer-bugs] [Bug 629349] [oggdemux] extract stream tags for tagreadbin and transcoding

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=629349
  GStreamer | gst-plugins-base | unspecified

sreerenj <bsreerenj> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bsreerenj at gmail.com

--- Comment #7 from sreerenj <bsreerenj at gmail.com> 2010-09-14 09:53:34 UTC ---
(In reply to comment #5)
> Re. the oggdemux patch, I think the tag extraction should be done in
> gstoggstream.c, since tags are in header packets, and header packets are parsed
> there. The existing ogm stuff in chain_peer should be moved there as well IMHO.
> 
> Moving to gst-plugins-base component for now, as gst-plugins is for old 0.8
> bugs and no one checks those.
> 
> On a side note, this is not just for tagreadbin, but useful in general, e.g.
> for stream extraction (flac audio into stand-alone flac) or transcoding.


Currently we are parsing only the identification header in gstoggstream.c. I
think we can add the comment header parsing for ogm,vorbis and vp8 to header
checking functions of each(is_header_ogm,is_header_vorbis and is_header_vp8) .

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.