[Mplayer-cvslog] CVS: main/libmpcodecs ve_lavc.c,1.5,1.6

[Arpi]

Hi,

>      /* this is just an extern from libavcodec but it should be in the
>         encoder context - FIXME */
> -    motion_estimation_method = lavc_param_vme;
> +    lavc_venc_context.me_method = lavc_param_vme;

imho this is a very Bad Idea.
it may be done with check in ./configure for libavcodec version...
but as Michael wrote to ffmpeg list, they should maintain compatibility
otherwise they break compatibility with many apps...
and mencoder won't work this change in libavcodec, as they renumbered
methods 0..5 to -4..1 or so...

conclusion: reversed.


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu