[FFmpeg-devel] [PATCH] move av_crc04C11DB7_update to libavutil
Måns Rullgård
mans
Mon Oct 29 11:05:34 CET 2007
Michael Niedermayer wrote:
> On Mon, Oct 29, 2007 at 09:03:39AM +0000, M?ns Rullg?rd wrote:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>>
>> > Hi
>> >
>> > On Sun, Oct 28, 2007 at 10:11:05PM +0100, Baptiste Coudurier wrote:
>> >> Aurelien Jacobs wrote:
>> >> > Michael Niedermayer wrote:
>> >> >
>> >> >
>> >> >>Hi
>> >> >>
>> >> >>On Sun, Oct 28, 2007 at 06:51:19PM +0100, Baptiste Coudurier wrote:
>> >> >>
>> >> >>>Hi
>> >> >>>
>> >> >>>$subject. Needed to avoid code duplication for new ogg muxer.
>> >> >>
>> >> >>libavutil is not supposed to be a place for code common between
>> libavcodec
>> >> >>and libavformat, even less so between 2 libavformat (de)muxers! but
>> rather
>> >> >>a small lib of generally usefull things av_crc04C11DB7_update() is not
>> >> >>usefull outside of libavformat
>> >> >>
>> >> >>so this function should stay in libavformat
>> >>
>> >> Humm, so it ok to add it to aviobuf.c and avio.h for now ?
>> >
>> > yes
>>
>> What sense does it make to have it in aviobuf.c?
>
> reducing code duplication between nut and ogg (de)muxers
What I meant was, what does this have to do with buffered I/O? Wouldn't
some other file be better suited for this function?
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list