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

[Paul B Mahol]

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.
>    driver ffmpeg
>    dll escape130
>    out YV12