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

[Oneric]

On Wed, Sep 09, 2020 at 15:28:05 +0200, Andreas Rheinhardt wrote:
> Oneric:
> > On Wed, Sep 09, 2020 at 01:02:04 -0500, rcombs wrote:
> >> -        .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.
> > 
> The ff_id3v2_mime_tags array (removed in patch 14 of this series)
> contains this. It has been added intentionally in
> f704eb612b3333a589d83741e07bfbdf1cffb8cb, yet without giving any reason
> for it.
> 
> - Andreas

Ohh, I missed it being in ff_id3v2_mime_tags, thanks. In that case 
disregard my previous mail. It's probably a good idea to keep "image/jpg" to 
not break any files, as long as "image/jpeg" stays the preferred option.