[MPlayer-dev-eng] [patch] lavc ME parameter handling cleanup & 2 little fixes

[Juan J. Sierralta P.]

On Sun, 2002-04-21 at 21:09, Rémi Guyomarch wrote:

> For libavcodec :
> - move Motion_Est_QTab[] to mpegvideo.c, since there's no need to
>   store it as many times as one include "avcodec.h" ;
> - use symbolic names instead of numbers in this array.

	I agree with these, I'll commit when I finish H.263+ AIC ...
	Thanks!
 
> For mplayer :
> - use Motion_Est_QTab[] from libavcodec so that the "vme" parameter is
>   now quality-oriented (0 = no ME, 1..5 = fastest to slowest, worst to
>   best). IMHO it's way more intuitive than the previous ordering ;
> - don't use 4MV with B frames as it's buggy right now.
> - don't use 4MV in "-pass 1" since it's not supported by lavc
>   (same problem as HQ mode). Symptoms : libavcodec refuse to allocate
>   context in MPV_encode_init() and mencoder crash at the fist call to
>   MPV_encode_picture().

	mencoder isn't my code, but as a user I prefer an order in the numbers
since the text parameters aren't allowed.
 
-- 
Juanjo sin .sig :(