Paul B Mahol (2018-12-09): > >> + if (st->codecpar->sample_aspect_ratio.num && > >> st->codecpar->sample_aspect_ratio.den) { > >> + av_reduce(&sc->aspect_ratio.num, &sc->aspect_ratio.den, > >> + st->codecpar->sample_aspect_ratio.num * > >> st->codecpar->width, > >> + st->codecpar->sample_aspect_ratio.den * > >> st->codecpar->height, INT_MAX);
> There is no av_reduce_q. But there are av_mul_q() and av_div_q(), that would make these computations much more readable, and are protected from overflow. Regards, -- Nicolas George
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel