[FFmpeg-devel] [PATCH]Write tiff aspect ratio by default instead of dpi

Carl Eugen Hoyos cehoyos at ag.or.at
Sat Apr 5 01:08:51 CEST 2014


Michael Niedermayer <michaelni <at> gmx.at> writes:

> > -    uint32_t res[2] = { s->dpi, 1 };    
> > // image resolution (72/1)
> > +    uint32_t resx[2] = { s->dpi ? s->dpi : 
> > avctx->sample_aspect_ratio.den, 1 };
> > +    uint32_t resy[2] = { s->dpi ? s->dpi : 
> > avctx->sample_aspect_ratio.num, 1 };
> 
> i think this is wrong
> shouldnt the dpi be adjusted when the aspect differs from 1:1

But isn't this unrelated to this patch?

Carl Eugen



More information about the ffmpeg-devel mailing list