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

Carl Eugen Hoyos cehoyos at ag.or.at
Sun Apr 6 03:43:38 CEST 2014


Clément Bœsch <u <at> pkh.me> writes:

> On Sat, Apr 05, 2014 at 02:52:37PM +0200, Carl Eugen Hoyos wrote:
> > On Saturday 05 April 2014 02:39:18 pm Clément Bœsch wrote:
> > > > +        AVRational y = av_mul_q((AVRational) { s->dpi, 1 },
> > >
> > >                                     ^^^^^^^^^^^^^^^^^^^^^^^^^
> > >                                     av_make_q()
> > 
> > How do the compilers mentioned in libavutil/rational.h 
> > compile libavcodec/mpeg12dec.c?
> > 
> 
> It's related to external apps I guess, I'm mentioning 
> av_make_q() because it makes code prettier™©®.

No objection (I was hoping the function would 
miraculously allow av_make_q(res) but that is 
unfortunately not possible...)

The patch was merged.

Thank you for the reviews, Carl Eugen



More information about the ffmpeg-devel mailing list