[gstreamer-bugs] [Bug 518857] parser baseclass

[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=518857

  GStreamer | gstreamer (core) | Ver: HEAD CVS

Sebastian Dröge changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #106082|0                           |1
        is obsolete|                            |
 Attachment #109805|0                           |1
        is obsolete|                            |




------- Comment #10 from Sebastian Dröge  2008-09-01 08:38 UTC -------
Created an attachment (id=117744)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=117744&action=view)
gstbaseparse.c

My latest version of gstbaseparse.c with all the diffs here merged adjusted for
flac (GstFlacBaseParse for the GType name). This fixes some bugs and improves
things at some places...

- Add a GstBaseParseClassPrivate for future extensions and add padding to
structs
- In pull mode, pull larger chunks and consume them to prevent a million 1-byte
pulls
- Always set the pad caps on outgoing buffers
- don't cache GST_EVENT_FLUSH_START
- Fix some timestamp calculations to not do calculations with
GST_CLOCK_TIME_NONE
- In pull mode, post an initial NEWSEGMENT event
- In pull mode, if the frame size is larger than the min_frame_size, pull the
complete frame instead of passing not enough data to the subclass


There are many more issues and possible improvements but I'll care for that
later ;)


-- 
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=518857.