[FFmpeg-devel] [PATCH 02/15] lavc/codec_desc: add additional JPEG and BMP MIME types

[Oneric]

On Wed, Sep 09, 2020 at 01:02:04 -0500, rcombs wrote:
> ---
>  libavcodec/codec_desc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/codec_desc.c b/libavcodec/codec_desc.c
> index 3dee640360..49a00ad264 100644
> --- a/libavcodec/codec_desc.c
> +++ b/libavcodec/codec_desc.c
> @@ -82,7 +82,7 @@ static const AVCodecDescriptor codec_descriptors[] = {
>          .name      = "mjpeg",
>          .long_name = NULL_IF_CONFIG_SMALL("Motion JPEG"),
>          .props     = AV_CODEC_PROP_INTRA_ONLY | AV_CODEC_PROP_LOSSY,
> -        .mime_types= MT("image/jpeg"),
> +        .mime_types= MT("image/jpeg", "image/jpg"),

"image/jpg" doesn't seeem to be an official IANA MIME type ("image/jpeg" is).
That ofc doesn't mean it isn't being used, but after a quick search I can't 
find a reference for "image/jpg" anywhere.
Did you encounter a program using "image/jpg"? If not it may be better to 
stick with "image/jpeg" only.


>          .profiles  = NULL_IF_CONFIG_SMALL(ff_mjpeg_profiles),
>      },
>      {
> @@ -588,7 +588,7 @@ static const AVCodecDescriptor codec_descriptors[] = {
>          .name      = "bmp",
>          .long_name = NULL_IF_CONFIG_SMALL("BMP (Windows and OS/2 bitmap)"),
>          .props     = AV_CODEC_PROP_INTRA_ONLY | AV_CODEC_PROP_LOSSLESS,
> -        .mime_types= MT("image/x-ms-bmp"),
> +        .mime_types= MT("image/x-ms-bmp", "image/bmp"),
>      },
>      {
>          .id        = AV_CODEC_ID_CSCD,
> -- 
> 2.27.0
>