[FFmpeg-devel] [PATCH 1/2] avutil/avstring: add av_strnlen()

Michael Niedermayer michaelni at gmx.at
Sun Dec 22 15:39:53 CET 2013


On Sun, Dec 22, 2013 at 12:14:12PM +0100, Giorgio Vazzana wrote:
> Hi Michael,
> 
> 2013/12/21 Michael Niedermayer <michaelni at gmx.at>:
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  libavutil/avstring.h |   11 +++++++++++
> >  1 file changed, 11 insertions(+)
> >
> > diff --git a/libavutil/avstring.h b/libavutil/avstring.h
> > index 882a2b5..fb8236f 100644
> > --- a/libavutil/avstring.h
> > +++ b/libavutil/avstring.h
> > @@ -132,6 +132,17 @@ size_t av_strlcat(char *dst, const char *src, size_t size);
> >  size_t av_strlcatf(char *dst, size_t size, const char *fmt, ...) av_printf_format(3, 4);
> >
> >  /**
> > + * Gets the count of continuous non zero chars starting from the begin.
> 
> Nit: maybe "...starting from the beginNING."?
> 
> > + */
> > +static inline size_t av_strnlen(const char *s, size_t len)
> > +{
> > +    int i;
> 
> How about size_t i; ?

both locally fixed

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131222/1be28fd3/attachment.asc>


More information about the ffmpeg-devel mailing list