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

[Corey Hickey]

Michael Niedermayer 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)

Sorry, I missed that one. This should be (finally) correct. Since I
think you're ok with them otherwise, I'll apply both patches when I wake
up tomorrow unless someone says "stop".

Thanks,
-Corey
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: sc_factor.ffmpeg.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20060122/7904c161/attachment.asc>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: sc_factor.mplayer.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20060122/7904c161/attachment.txt>