[gstreamer-bugs] [Bug 354451] [mythtvsrc] Plugin for MythTV source

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=354451

  GStreamer | gst-plugins-bad | Ver: HEAD CVS





------- Comment #21 from Rosfran Borges  2007-01-24 14:17 UTC -------
(In reply to comment #16)
>
>  - there are g_usleep (500); and sleep(9); statements and
>    similar in the code in the _start() function this is almost
>    always wrong and a sign that things should be designed
>    differently. Especially a sleep of 9 seconds is outright
>    evil, since it can't be interrupted, and an application
>    trying to interrupt it would block for at least that long
>    with the interface freezing up etc. Same for the normal
>    start-up case where this is triggered (arguably that's
>    a design problem in GstBaseSrc though, since it also
>    applies when doing open() on CD/DVD devices and the like).
> 

I removed these sleep calls in the new MythTV plug-in, and added these sync
calls to the library. Below, there is a patch to fix these issues. This patch
must to be applied in order to get the mythtvsrc element to properly run again
on LiveTV content.

Regards,
Rosfran Borges


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email