[Ffmpeg-devel] [PATCH] [RFC] Preparing for new grab interface

[Ramiro Polla]

Hello,

Alex Beregszaszi wrote:
> Hi,
>
>   
>> I'm writing a new grabbing selection interface. As a first step, attached
>> patch moves the #ifdef jungle from ffmpeg.c to the configure script. Also
>> allows the user to specify a default grab format in configure.
>>
>> Additionally, I plan to:
>> - Add a -gv option to specify video grab format (and -ga for audio). This
>> removes the need for x11: from the grab device, and moves it to "-gv
>> x11grab".
>> - Add a -gi option (with no args) to specify a grab input, much like the
>> -i option. So the device is treated as an input file at that point of the
>> command line, instead of having to be treated by prepare_grab. This way,
>> it can specify the size it wants just like a normal file, instead of
>> having to rely on -s for the output file.
>>
>> Please tell me if any this would be welcome, or if I'm just wasting my time.
>>     
>
> You need to separate your patches:
>
> 1, disabled() function for configure
> 2, configure selectable video grab format
>
>   
Ok then. Very small steps. Attached 1st patch.
> Also use dashes in the configure option. And I dont think
> --video-grab-format is the best option name available, maybe
> --default-video-grabber ?
>
>   
You're right. I'll change that.
> Also is that a good way disabling video4linux1 if video4linux2 is found?
> I know there is a wrapper layer for 1 when running on 2. For example I
> had now a extra-low-end camera, which didnt worked with V4L2, but
> started working when I tried V4L1 grabber in ffmpeg (I know, the driver
> should be fixed).
>   
I didn't really understand what you meant there. With this option you 
can specify the format, so you could easily specify v4l 1 instead of v4l2.

Ramiro Polla
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: disabled.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070122/02580cbc/attachment.asc>