[MPlayer-dev-eng] Another ATSC "a52: CRC check failed" problem

[Rich Felker]

On Tue, Nov 29, 2005 at 09:30:29AM +0100, Guillaume POIRIER wrote:
> Hi,
> 
> On 11/29/05, Scott W. Larson <scowl at pacifier.com> wrote:
> > Unlike my previous one, I think this is actually related to AC-3
> > packets. The clip to reproduce the problem (sent to me by someone else)
> > is here:
> >
> >     http://home.pacifier.com/~scowl/hdtv/foxksaz.ts
> >
> > This seems to be a problem in the "A52 WITH HEADER" part of the code in
> > libmpdemux/demux_ts.c. If I change the code to not skip over the header,
> > namely:
> >
> >     diff -r1.39 demux_ts.c
> >     1318,1319c1318,1319
> >     <                       es->start   = p+4;
> >     <                       es->size    = packet_len - 4;
> >     ---
> >     >                       es->start   = p;
> >     >                       es->size    = packet_len;
> 
> I don't know, all I can say is that we prefer "context diff" here (diff -u).

diff -u is _unified_ diff. context diff (diff -c) is something else
that we don't like.

rich