Hi,

Marc Mason wrote:
> Hello,
> 
> Consider an MPEG Audio Layer II elementary stream (test.mp2)
> 
> libavformat returns codec_id = CODEC_ID_MP3 not CODEC_ID_MP2
> 
> AFAIU, the layer is stored in sub_id
> 
> There is a comment :
> CODEC_ID_MP3 ///< preferred ID for decoding MPEG audio layer 1, 2 or 3
> 
> Shouldn't ff_mpa_decode_header set codec_id to CODEC_ID_MP2 when dealing 
> with a layer II ES ?
> 
> http://cekirdek.pardus.org.tr/~ismail/ffmpeg-docs/mpegaudio__parser_8c-source.html#l00047
> 
> Something along the lines of
> 
> $ svn diff
> Index: mpegaudio_parser.c
> ===================================================================
> --- mpegaudio_parser.c  (revision 16054)
> +++ mpegaudio_parser.c  (working copy)
> @@ -62,6 +62,7 @@
>           break;
>       case 2:
>           avctx->frame_size = 1152;
> +        avctx->codec_id = CODEC_ID_MP2;
>           break;
>       default:
>       case 3:
> 

You can ask this on ffmpeg-devel I think. Attach your patch.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org
_______________________________________________
libav-user mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/libav-user

Reply via email to