[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