[FFmpeg-devel] [PATCH 5/8] lavd: add device capabilities API

[Lukasz Marek]

On 05.04.2014 23:14, Michael Niedermayer wrote:
> On Sat, Apr 05, 2014 at 10:58:37PM +0200, wm4 wrote:
>> On Sat, 5 Apr 2014 22:47:37 +0200
>> Michael Niedermayer <michaelni at gmx.at> wrote:
>>
>>> On Sat, Apr 05, 2014 at 10:17:33PM +0200, Lukasz Marek wrote:
>>
>>>> I still think this there should be separate struct to store them in
>>>> both cases.
>>>> It can be moved to AVFormatContext to simplify API, but avformat
>>>> need to know size of the structure. It is avdevice specific so I
>>>> don't know if moving it to avformat is good idea. I can add alloc
>>>> function in libavdevice, but it will create dependency.
>>>> I don't know if it all is worth removing 2 simple API calls.
>>>
>>> well, if people are happy with how it is with teh 2 calls, i dont mind
>>
>> What people? Is anyone else working on / using the stuff discussed here?
>
> The API discussed here allows to get the capabilities of devices
> but they are technically muxers and demuxers, so maybe this API
> would at some point in the future also be used for gathering the
> capabilities of muxers and demuxers


I attached a bit updated version.
I separated sample format and pixel format (it had common field format). 
It could be a problem in case some device can handle both.

I'm almost done with implementation of the API for pulseaudio and opengl 
so it is quite well tested from developer point of view.

If there are no other remarks than you can merge this one.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavd-add-device-capabilities-API.patch
Type: text/x-patch
Size: 13104 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140409/8b488f47/attachment.bin>