Re: [FFmpeg-devel] [PATCH] nvenc: always reduce DAR width and height

2016-11-29 Thread Timo Rothenpieler
(avctx->sample_aspect_ratio.num != 1 || avctx->sample_aspect_ratio.num != 1)) { Damn, never noticed that typo. Just fixing the typo should be fine as well, but I like the new logic better so this LGTM and will push soon as well. ___ ffmpeg-devel mailing

[FFmpeg-devel] [PATCH] nvenc: always reduce DAR width and height

2016-11-26 Thread Miroslav Slugeň
This patch will fix/change: 1. avctx->sample_aspect_ratio.num == 1 and avctx->sample_aspect_ratio.den != 1: There is bug in old comparison, so with this aspect ratio for example 1/2 old alghoritm will produce aspect ratio 1/1 2. Old algorithm also does compute with negative numbers, which sho