[vlc-devel] [PATCH 2/3] OS/2: Include ctype.h in modules/demux/playlist/playlist.c
Måns Rullgård
mans at mansr.com
Fri Aug 24 23:49:30 CEST 2012
"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?
--
Måns Rullgård
mans at mansr.com
More information about the vlc-devel
mailing list