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

Michael Niedermayer michaelni
Sat Feb 19 16:01:22 CET 2011


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

[...]
--
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110219/9817fad2/attachment.pgp>



More information about the ffmpeg-devel mailing list