[Bug 701015] Using a g_thread for gnonlin pipeline update / forward eos on the streaming thread.

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=701015
  GStreamer | gnonlin | git

Thibault Saunier <thibault.saunier> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |thibault.saunier at collabora.
                   |                            |com

--- Comment #2 from Thibault Saunier <thibault.saunier at collabora.com> 2013-05-26 02:23:11 UTC ---
(In reply to comment #1)
> Thast is really cool stuff. Could you cleanup your branch, there is a lot do /
> undu, left over comment (even C++ style comment).

We know the history is not reaaly nice but we have been work both on that one
which leaded to what we have now.

> Aslo, starting a new thread each time might not be the best thing, discussion
> with Edward lead to the conclusion we should use a GstTask, and pause it when
> it's not needed. It will also ensure we never endup with two seeking threads
> (which would make no sense). Other then that, it's pretty nice that you take
> the time to look at those issues.

We are currently starting the thread between NULL to READY and killing it from
READY to NULL which iirc is the correct way of doing it.

We should never end up with various threads the way it is implemented, if it
happens, it is a bug that we need to fix.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.