[MPlayer-dev-eng] [PATCH] remove E130 from taglists

[Reimar Döffinger]

On Sun, Dec 11, 2011 at 05:59:10PM +0000, Paul B Mahol wrote:
> On 12/11/11, Reimar Doeffinger <Reimar.Doeffinger at gmx.de> wrote:
> > Hello,
> > this is now in FFmpeg taglists, so we should not need it.
> > Someone willing to test and apply if it works?
> 
> It is for escape130 video in avi
> (ffmpeg can create such samples with -vcodec copy).
> Escape130 video in rpl will not work.
> 
> > Index: libmpdemux/mp_taglists.c
> > ===================================================================
> > --- libmpdemux/mp_taglists.c	(revision 34422)
> > +++ libmpdemux/mp_taglists.c	(working copy)
> > @@ -84,7 +84,6 @@
> >      { CODEC_ID_DVVIDEO,           MKTAG('d', 'v', 's', 'd')},
> >      { CODEC_ID_EAC3,              MKTAG('E', 'A', 'C', '3')},
> >      { CODEC_ID_ESCAPE124,         MKTAG('E', '1', '2', '4')},
> > -    { CODEC_ID_ESCAPE130,         MKTAG('E', '1', '3', '0')},
> >      { CODEC_ID_FLV1,              MKTAG('F', 'L', 'V', '1')},
> >      { CODEC_ID_G729,              MKTAG('G', '7', '2', '9')},
> >      { CODEC_ID_H264,              MKTAG('H', '2', '6', '4')},
> > Index: etc/codecs.conf
> > ===================================================================
> > --- etc/codecs.conf	(revision 34422)
> > +++ etc/codecs.conf	(working copy)
> > @@ -3562,7 +3562,7 @@
> >  videocodec ffescape130
> >    info "FFmpeg Escape 130"
> >    status working
> > -  fourcc E130 ; internal MPlayer FourCC
> > +  fourcc E130
> +    format 0x82
> 
> adding that should make escape130 video in rpl works without problems.

No, the current solution is better, I missed/forgot it was in the
override list.