[MPlayer-dev-eng] Enhanced cropdetect code

[Roger Heflin]

compn wrote:
> On Thu, 24 Jul 2008 20:28:18 -0500, Roger Heflin wrote:
>> What about adding a 3rd option onto cropdetect that allows skipping of frames ie
>> -vf cropdetect=0:12:30 says only check every 30th frame?
> 
> there is -vf framestep=30 or -sstep for skipping frames
> in which case cropdetect will redo its calculations and print new numbers.
> (i think this is in the docs)
> 
> -compn

Yes they are in the Docs, but....

Neither of those options currently appear to work with cropdetect (the options 
are accepted-but don't work as expected).   Other have hinted at there being an 
obvious reason why -ss won't work with cropdetect, but I don't see any possible 
*real* reason why they won't work together except a coding choice making it not 
easy to implement.

In the -ss case the option is accepted, but does not appear to actually cause 
cropdetect to skipframes, if you use the framestep option it is accepted and 
works but cause cropdetect to not be done ( -vf frameskip=30,cropdetect=12), if 
you use (-vf cropdetect=12,frameskip=30 ) then cropdetect is done but no 
frameskip appears to be happening.

                              Roger