[mythtv] VideoSync class

[Mark Kendall]

On Thu, 9 Jan 2020 at 18:34, Piotr Oniszczuk <piotr.oniszczuk at gmail.com> wrote:
> Is it possible to add back old sync method for some time just for testing purposes?
>
> Reasons:
> 1. new method has some side effects for me.
> Namely:
> -on nvidia playback is smooth but I can notice with very little (but noticeable) micro jumps (best seen on new scrolling bars).
> They are random - every few to 10sec - but noticeable.
> Old sync hasn't this effect - so it will be good to compare and make sure new method not causing this issue.
> -on stateful decoders (amlogic) playback sometimes enters jumpy video (audio is still ok). To fix - i need to exit playback.
> I’m not sure where issue is - but it will be good to exclude new sync root cause here.
>
> 2.Good test for new sync method is not regressing is to make sure all sync playback issues (if present) - are manifesting on both methods.
> This will make us sure that new method not regressing when compared to old one in corner cases….

Piotr

Adding back old code is not a priority for me. Time is currently
limited for development work so I would rather focus on fixing issues
rather than trying to work out whether they worked before or not.

And you seem confident that the old sync worked anyway?

I'd rather see some playback logs and understand if the issues are
repeatable and fixable.

Thanks and regards
Mark