[mythtv] [patch] Fix for sluggish pause

David Engel gigem at comcast.net
Sat Dec 4 04:25:44 UTC 2004


On Wed, Dec 01, 2004 at 10:29:38PM -0600, David Engel wrote:
> On Wed, Dec 01, 2004 at 09:50:56AM -0600, David Engel wrote:
> > I'm going to look at the speed change needs again and see if I can do
> > something without a mutex that won't have race conditions.
> 
> Just a quick update.  I've made some progress, but it will probably be
> another day or two before I'm finished.

Here's my first cut.  It basically moves nearly all of the pause and
speed change logic into one place in the decoder loop.  It could go
pretty much anywere in the loop but the very top seems most logical.

I've only done limited testing so far.  I will do more this weekend,
but it should be close enough for others to provide feedback.  There
might be a very tiny race condition, but I think it should be
inconsequential, and if it turns out not to be, it will hopefully be
much easier to fix now.

Finally, I have chosen to ignore the waitvideo paramter to NVP::Pause
and the unpauseaudio parameter to NVP:Play for now.  I'm need to look
at how and why they are needed before reimplementing them.

David
-- 
David Engel
gigem at comcast.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: speed.patch.gz
Type: application/octet-stream
Size: 2465 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20041203/d572134d/speed.patch.obj


More information about the mythtv-dev mailing list