[FFmpeg-devel] [PATCH 2/3] lavf: replace all uses of url_fskip with avio_seek

Luca Barbato lu_zero
Wed Mar 2 15:58:32 CET 2011


On 03/02/2011 02:29 PM, Michael Niedermayer wrote:
>> We considered avio_skip() as a macro for avio_seek(SEEK_CUR), but
>> Anton preferred this variant because it was clearer to the API user
>> what was going on. It's his patch, therefore he decides.
> 
> unless you can read peoples minds, i dont think you can know what the API
> users consider cleaner.

That goes both side.

> also, if its antons patch he decides
> if its my patch (you remember last i send or even every i send since you
> hijaked the repository) its suddenly your decission and not mine

Do you have a patch to provide avio_skip() as macro to avio_seek? I
guess that's up to Anton put the macro or not in HIS patch, yet anybody
could propose to add quick macros.

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list