[FFmpeg-devel] [PATCH] avio API cleanup II

[Ronald S. Bultje]

Hi,

On Sat, Feb 19, 2011 at 10:01 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Sat, Feb 19, 2011 at 12:42:44PM +0100, Anton Khirnov wrote:
>> Hi,
>> due to BBB being lazy, here's the updated batch of renames.
>>
>> The basic idea is that functions working on AVIOContext (formerly
>> ByteIOContext) get an avio_ prefix, while functions working on
>> URLContext (to be renamed to AVURLContext) get an av_url_ prefix.
>>
>> Comments welcome.
>
> You are making hundreads of internal functions part of the public API
> The correct prefix is ff not av for quite a few i think
> That said if you make things part of the public API have you thought about
> the structs being ready for this? Can they be extended without breaking ABI?
>
> old team: technical reviews
> new team: political reviews because they plain dont know they code for which
> ? ? ? ? ?they review, the people knowing have been kicked out

No, tech reviews simply take more time so I don't have them ready in
less than a minute.

Ronald