[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