[FFmpeg-soc] [soc]: r5877 - in mms: mms.c mms.h mmsh.c mmst.c

Benoit Fouet benoit.fouet at free.fr
Mon Aug 2 10:02:49 CEST 2010


On Sun, 1 Aug 2010 14:22:22 -0400 Ronald S. Bultje wrote:
> Hi,
> 
> On Sun, Aug 1, 2010 at 3:51 AM, spyfeng <subversion at mplayerhq.hu> wrote:
> > Modified: mms/mms.c
> > ==============================================================================
> [..]
> > +int ff_read_data(MMSContext *mms, uint8_t *buf, const int buf_size)
> 
> this will be a public symbol, please make it a little more specific,
> ff_mms_read_some_kind_of_data(). Same for the other function (make it
> ff_mms_asf_header_parser()).
> 

BTW, what is the purpose of making an 'int' parameter 'const'?

Ben


More information about the FFmpeg-soc mailing list