[FFmpeg-devel] [PATCH 1/2] lavd/avdevice: replace av_ with ff_ for caps option table

[Lukasz Marek]

On 13.12.2014 20:48, Reimar Döffinger wrote:
> On 13.12.2014, at 20:42, James Almer <jamrial at gmail.com> wrote:
>> On 13/12/14 4:27 PM, Lukasz Marek wrote:
>>> This symbol is required for query capabilities callbacks.
>>> This symbol is only required by libavdevice and should not be exported.
>>
>> This need a deprecation and an FF_API scheduled removal. We can't remove the symbol
>> without a major bump.
>
> Well, there is a comment saying it should not be used (which raises the question why it was placed in that header though).
> If a search can't find anyone using it, it might not be _completely_ unreasonable to cheat and remove it without a major bump.
> But I have no strong opinion either way.

Yes. There is a comment about that it is internal. So I don't think it 
is serious to rename it. We already renamed symbols, but probably in not 
exported headers though.

Unless user created own device and implemented capabilities query, it is 
not used yet. It is not used in ffmpeg project. I'm about to push first 
implementations so I wanted to fix it before, as spotted.

>> Also, an internal.h header for stuff like this is probably a good idea.
>
> Yes.

OK, I will move it.