After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 679729 - Buffer-list functionality is missing in multiudpsink
Buffer-list functionality is missing in multiudpsink
Status: RESOLVED NOTABUG
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
0.11.x
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-07-11 07:40 UTC by Patricia Muscalu
Modified: 2012-07-13 07:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
added buffer-list support in multiudpsink (10.92 KB, patch)
2012-07-11 07:40 UTC, Patricia Muscalu
none Details | Review

Description Patricia Muscalu 2012-07-11 07:40:03 UTC
Created attachment 218499 [details] [review]
added buffer-list support in multiudpsink

Buffer-list support has been removed from multiudpsink.
The attached patch re-enables buffer-list support in multiudpsink.
The patch re-enables udpsink unit tests as well.
Comment 1 Wim Taymans 2012-07-11 08:00:41 UTC
> Buffer-list support has been removed from multiudpsink.

This was intentional because the default bufferlist handling in basesink
should be enough. Am I missing something?
Comment 2 Patricia Muscalu 2012-07-11 10:50:07 UTC
Ok, now I understand how it works ;-). Basically, the virtual render_list function can/will be removed from basesink, correct?

Ignore my patch. Sorry for the inconvenience!
Comment 3 Tim-Philipp Müller 2012-07-11 10:58:36 UTC
> [T]he virtual render_list function can/will be removed from
> basesink, correct?

I think this is true in this particular case, but there are still scenarios where the render_list function is useful, e.g. in multifilesink when buffer lists are used to group a GOP (and you only want to write a whole GOP to file).
Comment 4 Patricia Muscalu 2012-07-11 11:04:09 UTC
I understand. Thank you for your answer.