Include file cleanup : dce.h

Rémi Assailly remi.assailly at free.fr
Tue Dec 14 05:38:25 CST 2004


Alexandre Julliard wrote :

>Rémi Assailly <remi.assailly at free.fr> writes:
>
>  
>
>>Changelog:
>>* get rid of dce.h
>>--- dlls/x11drv/winpos.c	2004-12-09 19:10:35.000000000 +0100
>>+++ dlls/x11drv/winpos.c	2004-12-11 19:46:03.000000000 +0100
>>@@ -40,7 +40,6 @@
>> #include "x11drv.h"
>> #include "win.h"
>> #include "winpos.h"
>>-#include "dce.h"
>> #include "cursoricon.h"
>> #include "nonclient.h"
>> 
>>@@ -49,6 +48,8 @@
>> 
>> WINE_DEFAULT_DEBUG_CHANNEL(x11drv);
>> 
>>+extern BOOL DCE_InvalidateDCE( HWND, const RECT* );
>>    
>>
>
>The goal of the header files cleanup is not to mechanically remove the
>#includes by duplicating the definitions, if we wanted to do that it
>would have been done long ago. The real goal is to get rid of the
>inter-dll dependencies, the global headers are really just a symptom
>of the problem.
>  
>
Well, I see.
Do you agree if I first move all definitions used by only one dll ? ( in 
the case of dce.h, we'll just keep
extern BOOL DCE_InvalidateDCE( HWND, const RECT* ) )
What will stay is inter-dll dependencies and we'll be sure the problem 
is here and we'll work on it.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.winehq.org/pipermail/wine-devel/attachments/20041214/642f34e8/attachment.html


More information about the wine-devel mailing list