[Ffmpeg-cvslog] CVS: ffmpeg/libavcodec motion_est.c,1.115,1.116

[Michael Niedermayer]

Hi

On Sat, Jan 21, 2006 at 01:38:39PM -0800, Corey Hickey wrote:
[...]
> > except these the patch is ok and can be applied
> 
> Does this pair of patches look ok? I want to be sure I'm not about to
> break something on my first ffmpeg commit. :) Differences:
> 
> * sc_factor=1 is default
> * INT_MAX is the maximum value
> * all sc_factor lines are at the bottom, below other parameters
> * mplayer man page description is a bit more detailed

[...]
> +    int scenechange_factor;
> +
> +    /**
>       * minimum lagrange multipler
>       * - encoding: set by user.
>       * - decoding: unused

ok, strictly this doesnt break ABI compatibility due to AVOptions but i
really would prefer if it where at the end (and iam sure everyone who
doesnt use AVOptions would be happier too)


[...]

-- 
Michael