[FFmpeg-devel] [PATCH] avcodec/snowenc: fix setting motion_est option

[Michael Niedermayer]

On Fri, Aug 25, 2017 at 09:07:53PM -0300, James Almer wrote:
> Remove usage of FF_MPV_COMMON_OPTS, and set SnowContext.motion_est directly.
> Based on code from svq1enc.c
> 
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavcodec/snow.h    | 3 ++-
>  libavcodec/snowenc.c | 9 +++++----
>  2 files changed, 7 insertions(+), 5 deletions(-)

I tested this with some ME options and it seems to break encode

./ffmpeg -i ~/videos/matrixbench_mpeg2.mpg  -an -vcodec snow -vf scale=160:120 -t 1 -refs 3 -me_method iter -dia_size 2 -cmp 12 -subcmp 12 test.avi

There are blinking gray images



[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170826/577f9187/attachment.sig>