[mythtv-users] Re: Time stretch bug in 0.18

Chris Pinkham cpinkham at bc2va.org
Mon Apr 25 01:09:46 UTC 2005


> On Sun, Apr 24, 2005 at 04:49:25PM -0400, Chris Pinkham wrote:
> > If you're compiling from source, can you try a simple 1-line fix?
> > 
> > Edit libs/libmythtv/tv_play.cpp and search for the following line:
> > 
> > 	int speedX10 = (int)(ceil(normal_speed * 10));
> > 
> > change the 'ceil' to 'round' so it looks like this:
> > 
> > 	int speedX10 = (int)(round(normal_speed * 10));
> > 
> > and see if that fixes the problem for you.
> 
> That will probably work for the menu-defined speeds which are all
> multiples of 0.10.  I don't think it will work right, however, if the
> user does a manual adjustment in 0.05 increments to an in-between
> speed.  IMO, the menu should not show a checkmark for any speed in
> those cases.
> 
> David

Maybe I should have made that a speedX100 instead then.  I wasn't
thinking about the manual adjustments by 0.05.  If none of the
(speedX10 == blah) checks match, then nothing is checked, so that
would accomplish what your describing if I switched to speedX100.
Unless you or someone has a better way to be checking the
normal_speed value without having to have something like:

	(normal_speed > 0.49 && normal_speed < 0.51)

I thought that those would be too complicated and that's why
I did the speedX10 to begin with.

Does switching to speedX100 vs speedX10 make sense in addition to
the round vs ceil change?
-- 
Chris



More information about the mythtv-users mailing list