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

Arpi arpi at thot.banki.hu
Sun Apr 14 22:39:12 CEST 2002


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



More information about the MPlayer-cvslog mailing list