[FFmpeg-soc] [soc]: r2900 - mxf/mxfenc.c

zhentan feng spyfeng at gmail.com
Wed Jul 30 17:03:56 CEST 2008


Hi

2008/7/30 Benoit Fouet <benoit.fouet at purplelabs.com>

> Hi,
>
> spyfeng wrote:
> > Author: spyfeng
> > Date: Tue Jul 29 18:28:24 2008
> > New Revision: 2900
> >
> > Log:
> > remove utf8len() and use strlen() instead.
> >
> >
> > Modified:
> >    mxf/mxfenc.c
> >
> > Modified: mxf/mxfenc.c
> >
> ==============================================================================
> > --- mxf/mxfenc.c      (original)
> > +++ mxf/mxfenc.c      Tue Jul 29 18:28:24 2008
> > @@ -450,9 +440,9 @@ static int mxf_write_identification(AVFo
> >
> >      put_buffer(pb, klv->key, 16);
> >
> > -    company_name_len = utf8len("FFmpeg") + 1;
> > -    product_name_len = utf8len("OP1a Muxer") + 1;
> > -    version_string_len = utf8len(LIBAVFORMAT_IDENT) + 1;
> > +    company_name_len = strlen("FFmpeg") + 1;
> > +    product_name_len = strlen("OP1a Muxer") + 1;
> > +    version_string_len = strlen(LIBAVFORMAT_IDENT) + 1;
> >
>
> why not using sizeof, instead of a function call ?
> strlen("abcd") + 1 == sizeof("abcd")
>

yes, it's simple use sizeof().
modified.
-- 
Best wishes~



More information about the FFmpeg-soc mailing list