[vlc-devel] Patch: New compat function strtoi(3)

Kamil Rytarowski n54 at gmx.com
Fri Feb 6 11:41:12 CET 2015


Me wrote:
> Rémi Denis-Courmont wrote:
> > Le 2015-01-20 20:48, Kamil Rytarowski a écrit :
> > > Locally defined function strtoi in src/config/file.c overlaps with
> > > strtoi(3) [1] from NetBSD.
> > >
> > > I'm attaching a patch adding NetBSD's flavour of the function to
> > > compat/, instead of wrapping our own.
> > 
> > The compat stuff seems OK. The rest seems wrong: it changes the 
> > semantics of the configuration parser. It would be far easier to just 
> > rename the static function.
> > 
> 
> I'm OK with both solutions. Possible new function name: strtoi() -> vlc_strtoi().

Hello.

Any update?
In NetBSD we went for rename strtoi() -> vlc_strtoi().
Here is the patch:
http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/multimedia/vlc21/patches/patch-src_config_file.c?annotate=1.1



More information about the vlc-devel mailing list