[gstreamer-bugs] [Bug 584465] basetransform does unneeded pad_allocs

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

  GStreamer | gstreamer (core) | Ver: git

Stefan Kost (gstreamer, gtkdoc dev) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #135744|reviewed                    |none
               Flag|                            |




------- Comment #3 from Stefan Kost (gstreamer, gtkdoc dev)  2009-06-01 16:07 UTC -------
I still have to explanation for the remaining 1382. I added some logging and
get a lot of elements with same_output=1, but the pad_alloced buffer is not
passed as an input:
<gain_0x8448198> same_output=1 last_buffer=0xb2936820 in_buf=0xb2902d58

the upstream called pad_alloc for volume's sink pad, but then pushes a
different buffer. It might be caused be other unneeded pad_allocs from
upstream. Adding a hashtable to track all pad_allocs and drop the ones which
are seen in prepare output buffer sounds expensive.


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