[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