[Gstreamer-bugs] [Bug 143030] - [PATCH] improve gstplay

[bugzilla-daemon at bugzilla.gnome.org]

http://bugzilla.gnome.org/show_bug.cgi?id=143030
GStreamer | gst-plugins | Ver: HEAD CVS





------- Additional Comments From julien at moutte.net  2004-05-24 11:15 -------
Well basically you brought back the GST_PLAY_TYPE that has been kicked out. The
reason i removed that and used a single pipeline was to be able to just handle
all cases with a single pipeline.

For audio only medias the video part of gstplay is not even connected by spider
so no data flows into it (no overhead). Visualization was added removed on the
fly depending on what users wanted.

Current libgstplay runs in it's own thread and you just moved the g_idle_add
from the player.c file directly inside your gstplay object, i don't really see
how it was not usable in KDE before...

I don't really mind to just replace current libgstplay by a rewrite from scratch
but basically for me that one is just bringing back the GST_PLAY_TYPE that i
found so ugly before. The only discussion i can see about all that stuff is
wether or not applications have to do gst_play_new
(GST_PLAY_MODE_VIDEO_WITH_VISUALIZATION, &error) or just gst_play_new (&error)
and let the pipeline accomodate itself to the media flowing inside.

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
You are the QA contact for the bug, or are watching the QA contact.