[FFmpeg-devel] [PATCH] avcodec/mjpegenc_common: always store JFIF, idependent of the aspect ratio

Reimar Döffinger Reimar.Doeffinger at gmx.de
Wed Jul 16 08:18:58 CEST 2014



On 15.07.2014, at 16:31, Michael Niedermayer <michaelni at gmx.at> wrote:

> Found-by: kriegero1
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

Nit: iNdependent in commit message.

> +    /* JFIF header */
> +    put_marker(p, APP0);
> +    put_bits(p, 16, 16);
> +    avpriv_put_string(p, "JFIF", 1); /* this puts the trailing zero-byte too */
> +    put_bits(p, 16, 0x0102);         /* v 1.02 */
> +    put_bits(p,  8, 0);              /* units type: 0 - aspect ratio */

Would an empty JFIF header possibly help as well? (I think it's a collection of tags or am I wrong and it requires an aspect entry?)
I don't like specifying a 1:1 aspect explicitly when we actually don't know it.
Also, might the diff be more readable without reindenting?


More information about the ffmpeg-devel mailing list