[mythtvnz] mythtv 0.25 jitter/judder

[Jean-Yves Avenard]

On Thursday, 14 June 2012, Brett wrote:

> On Fri, 2012-06-15 at 01:31 +1000, Jean-Yves Avenard wrote:
> > On 14 June 2012 20:07, Brett <mythicalbeast at slingshot.co.nz<javascript:;>>
> wrote:
> > > The commit related to refreshrate vs. frame_interval was
> > >
> >
> > you're saying that by reverting commit
> > 8fd44c8f6b0c7c2053a61c2590c781961023c8bb you get smooth playback?
> >
> By reverting parts of it..yes.
> The reverting changes on lines: 1947, 1949, 1950 & 1956 required.
> Line 1947 is crucial.
>
> I'm experimenting to see the effects of the other variable changes.
>
> The calculation on line 1979 & 1804 has changed w.r.t. mythtv0.24+fixes
> // 0.25+fixes
> if (avsync_used > refreshrate)
>                {
>                    avsync_adjustment += refreshrate;
>                }
>                else if (avsync_used < 0 - refreshrate)
>                {
>                    avsync_adjustment -= refreshrate;
>                }
>
> //0.24+fixes
> if (avsync_avg > frame_interval * 3 / 2)
>                {
>                    avsync_adjustment += refreshrate;
>                    lastsync = true;
>                }
>                else if (avsync_avg < 0 - frame_interval * 3 / 2)
>                {
>                    avsync_adjustment -= refreshrate;
>                    lastsync = true;
>                }
> // far as I can tell refreshrate = frame_interval at playback speed x1..
>
> Brett
>
>
>
I have no idea what line you are referring to. What file you are talking
about and which branch of mythtv you are referring to.
I'm guessing libs/libmythtv/mythplayer.cpp.
Line 1947 with no context means nothing to me. Especially as none of the
line numbers you've mentioned have anything to do with the code you quoted

If you want me to have a look at your problem, you'll need to be a little
bit more clear.
If you have a patch, provide a patch.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ourshack.com/pipermail/mythtvnz/attachments/20120615/2c916a12/attachment.html>