[FFmpeg-devel] [PATCH] move av_crc04C11DB7_update to libavutil

Måns Rullgård mans
Mon Oct 29 10:03:39 CET 2007


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?

> though maybe they should have a ff_ prefix to indicate that they are
> not part of the public API instead of av_

Yes.  IMHO, we should be more strict about following these naming
conventions.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list