[FFmpeg-devel] [PATCH] lavu: add av_strtok_r()

Stefano Sabatini stefasab at gmail.com
Sat Oct 15 21:01:27 CEST 2011


On date Saturday 2011-10-15 15:45:02 +0200, Michael Niedermayer encoded:
> On Sat, Oct 15, 2011 at 12:50:11PM +0200, Stefano Sabatini wrote:
> > The function strtok_r() is part of the POSIX.1 specification, but is not
> > available on some platforms. We provide an internal implementation, so we
> > do not need to rely on a platform implementation.
> > 
> > The code is based on the OpenBSD libc strtok_r() implementation.
> 
> LGTM and great idea

I rewrote the function from scratch, which reads more readable to my
eyes (also I was missing the original copyright notice, which should
not be anymore required).

I also changed the docs, to make the doxy more compliant with the
official POSIX.1 strtok_r() definition, which is explicitely referenced.

I'll push it in a day or so if I read no more comments.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0005-lavu-add-av_strtok_r.patch
Type: text/x-diff
Size: 7695 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111015/a498fbf1/attachment.bin>


More information about the ffmpeg-devel mailing list