[vlc-devel] [PATCH 2/3] OS/2: Include ctype.h in modules/demux/playlist/playlist.c
remi at remlab.net
Sun Aug 26 19:12:54 CEST 2012
On Saturday 25 August 2012, Måns Rullgård wrote:
> "Rémi Denis-Courmont" <remi at remlab.net> writes:
> > Le vendredi 24 août 2012 17:17:25 Måns Rullgård, vous avez écrit :
> >> ctype.h is a standard header so keep things simple and include it
> >> unconditionally.
> > The code that needs <ctype.h> in this file is _not_ standard. It is
> > Windows- specific.
> Apparently OS/2 needs it too.
> > I don't see the need to include the headers on all platforms.
> What harm does it do?
I don't think there is harm either way. I just like to not include unneeded
However, isalpha() on POSIX depends on the locale. And that would be incorrect
in this context: the affected code wants to check for ASCII letters only.
Perhaps <ctype.h> should be removed altogether there.
More information about the vlc-devel