[gstreamer-bugs] [Bug 434824] New: rtp mp4 payloader doesn't handle newsegments and flush events correctly

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=434824

  GStreamer | gst-plugins-good | Ver: HEAD CVS
           Summary: rtp mp4 payloader doesn't handle newsegments and flush
                    events correctly
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: sjoerd at luon.net
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


Hi,

  The rtpmp4vpayloader does some buffering of it's own to group several parts
of
  the mpeg stream into one RTP packet. But unfortunately it has no code to
handle
  new segments or flushes..

  I've got an application concatinates streams saved in multiple file into one
  seamless rtp stream, which obviously causes a new segment as soon as a new
file
  is started. 

  Because of the buffering, it can happen that a new segment is sent downstream 
  before the last buffer timestamped relative to that segment travels
downstream.   
  Which obviously causes issues.. So the cache needs to be flushed out before   
  sending the new segment event further downstream.

  And on a flush stop event the cache should be obviously just be emptied..

  I've solved this by overriding the event function of the basertppayloader as
it
  doesn't have the functionality. So a proper event virtual function might be a 
  nice addition to the base payloader. OTOH a quick grep showed that 6 of the
16 
  payloaders in -good use a similar buffering scheme, so it might be usefull 
  to move that to the base payloader..


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=434824.