On 08/07/2012 11:31 PM, Mohammad Alsaleh wrote:
> id3 v2.2 uses image format ("JPG","PNG") instead of mimetypes.
> 
> Currently, the attached picture is skipped because the format string
> does not match a known picture mimetype.
> 
> This patch fixes this behaviour.
> 
> Signed-off-by: Mohammad Alsaleh <[email protected]>
> ---
>  libavformat/id3v2.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c
> index 7a229d2..903198d 100644
> --- a/libavformat/id3v2.c
> +++ b/libavformat/id3v2.c
> @@ -114,8 +114,10 @@ const char *ff_id3v2_picture_types[21] = {
>  const CodecMime ff_id3v2_mime_tags[] = {
>      {"image/gif" , AV_CODEC_ID_GIF},
>      {"image/jpeg", AV_CODEC_ID_MJPEG},
> +    {"JPG",        AV_CODEC_ID_MJPEG}, /* v2.2  */
>      {"image/jpg",  AV_CODEC_ID_MJPEG},
>      {"image/png" , AV_CODEC_ID_PNG},
> +    {"PNG" ,       AV_CODEC_ID_PNG}, /* v2.2  */
>      {"image/tiff", AV_CODEC_ID_TIFF},
>      {"image/bmp",  AV_CODEC_ID_BMP},
>      {"",           AV_CODEC_ID_NONE},
> 

Looks fine, Diego do you prefer the list ordered?

lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to