rpcrt4: Fix some conditional includes

Alexandre Julliard julliard at winehq.org
Fri May 12 17:01:41 CDT 2006


Francois Gouget <fgouget at free.fr> writes:

> On Fri, 12 May 2006, Alexandre Julliard wrote:
> [...]
>> There's no point, since we haven't yet found a platform that doesn't
>> have them. The defines only exist because of things that autoconf does
>> behind the scenes, we don't explicitly check for these headers.
>
> And yet there are some places where we check for HAVE_SYS_TYPES_H. And
> if autoconf says these should be conditional there's no harm in
> checking, that way when that platforms turns up we'll be ready.

There are plenty of headers that don't have ifdefs, and we shouldn't
add them if there's no reason to, it just clutters up the code. If the
ifdef really becomes necessary one day, it's easy enough to add it
then.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list