[MPlayer-dev-eng] [PATCH] Fix for fix for mpeg2 A/V sync bug.

[Erik Auerswald]

Hi,

On 10/22/2011 06:25 PM, Dan Oscarsson wrote:
> In August we removed an older fix for bad pts values on video as it
> resulted in bad a/v sync when playing NTSC DVDs. Nobody them remembered
> why it was there.
>
> Recently I sent an updated mplayer with my patches added to a friend who
> plays a lot of mpeg-ts files gotten from DVB-t TV streams. And they no
> longer played correctly. I checked and now I remember that about a year
> ago I had a patch for this, but we never included my patch instead the
> above patch we removed, fixed it.
>
> The problem is that when playing a mpeg-ts file with h264 in, pts values
> are not reordered as demux_ts do not set correct-pts. This results in
> pts values not being in order.
>
> As the above removed patch had problems with NTSC DVDs I suggest we
> instead we patch a modified version of my old patch.
>
> The attached patch enables so re-order of pts values are always done.
> I cannot think of when this should be a problem. Looks good in all my
> tests.

I have tested your patch with a PAL DVD (Sucker Punch). Your patch 
introduces an audio delay (audio is too late) not present without the 
patch. This delay seems to increase during movie playback, it becomes 
obvious a few minutes after start.

Maybe a better way to handle the DVB-T MPEG-TS streams issues is to just 
specify -correct-pts?

Regards,
Erik