Hi

On Mon, Aug 2, 2010 at 4:02 PM, Benoit Fouet <[email protected]> wrote:

> On Sun, 1 Aug 2010 14:22:22 -0400 Ronald S. Bultje wrote:
> > Hi,
> >
> > On Sun, Aug 1, 2010 at 3:51 AM, spyfeng <[email protected]> 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'?
>
> hmm, usually, const will be used for a pointer.
here, It means buf_size is read only.
zhentan
-- 
Best wishes~
_______________________________________________
FFmpeg-soc mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc

Reply via email to