[PATCH] activeds: Exclude unused headers.

Amine Khaldi amine.khaldi at reactos.org
Tue Mar 12 06:12:40 CDT 2013


On 12/03/2013 11:27, Alexandre Julliard wrote:
> Amine Khaldi <amine.khaldi at reactos.org> writes:
>
>> @@ -28,13 +28,10 @@
>>  
>>  #include "windef.h"
>>  #include "winbase.h"
>> -#include "winuser.h"
>>  
>>  #include "objbase.h"
>>  #include "iads.h"
>> -#include "adshlp.h"
>>  
>> -#include "wine/unicode.h"
>>  #include "wine/debug.h"
> It's not worth the trouble, particularly since the file ends up
> including windows.h anyway.
>
In this case ending up with windows.h is not needed, so if we:

#define WIN32_NO_STATUS
#define _INC_WINDOWS
#define COM_NO_WINDOWS_H

That prevents it.

Should I send a second patch with these defines ?

Regards,
Amine.



More information about the wine-devel mailing list