Fix ntdll compilation in Darwin

Dmitry Timoshkov dmitry at baikal.ru
Thu Jun 30 09:04:04 CDT 2005


"Phil Krylov" <phil at newstar.rinet.ru> wrote:

>> > #ifdef SENSEBUFLEN
>> >     if (pPacket->SenseInfoLength > SENSEBUFLEN)
>> > +#elif defined( __APPLE__ )
>> > +    if (pPacket->SenseInfoLength > kSenseDefaultSize)
>> > #else
>> >     if (pPacket->SenseInfoLength > sizeof(struct request_sense))
>> > #endif
>> 
>> Please do not introduce even more platform dependent #ifdefs into cdrom.c,
>> currently it already has enough mess with all that #if defined(linux)/(__FreeBSD__)/
>> (__NetBSD__). Add proper configure checks for headers and structures your
>> platform needs.
> 
> To be sure I get you right: would changing __APPLE__ to HAVE_IOKIT (and
> adding HAVE_IOKIT detection to configure) be sufficient?

I'd assume it's enough, yes. kSenseDefaultSize requires another test I think.

-- 
Dmitry.




More information about the wine-devel mailing list