<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<HTML>
<HEAD><TITLE></TITLE>
<STYLE>
body, table, tr, td, p {font-family: Verdana;font-size:12px;margin: 0px 0px 0px 0px}
.bgtabl {BACKGROUND-REPEAT: no-repeat}
</STYLE>
</HEAD>
<BODY bgProperties="fixed" bgcolor="#FFFFFF" background="http://flashimg.club-internet.fr/flashmail/vide.gif">
<br>
Hi Alexandre,<br>
<br>
At least for directx headers it's not a good idea.<br>
many old games (using old directx) need to have compatible headers<br>
else they won't build (it's why bug <a href="http://winehq.com/hypermail/wine-patches/2005/03/att-0200/01-includes_and_bug2483.diff">2483 exists: build problem on an old dx game)</a><br>
<br>
And as for directx &lt; 8, microsoft used to reuse same headers name we need to flag <br>
<br>
But for classic windows headers i'm ok with you<br>
<br>
Regards,<br>
Raphael<br>
<br>
<br>----Message d'origine----
<br>&gt;A: Raphael <fenix @club-internet.fr="">
<br>&gt;Copie �: wine-devel@winehq.org
<br>&gt;Sujet: Re: [DMUSIC/DINPUT] includes fixes
<br>&gt;De: Alexandre Julliard <julliard @winehq.org="">
<br>&gt;Date: 11 Mar 2005 11:04:30 +0100
<br>&gt;
<br>&gt;Raphael <fenix @club-internet.fr=""> writes:
<br>&gt;
<br>&gt;&gt; --- dinput.h        5 Oct 2004 04:38:15 -0000        1.46
<br>&gt;&gt; +++ dinput.h        10 Mar 2005 22:13:43 -0000
<br>&gt;&gt; @@ -640,7 +640,9 @@
<br>&gt;&gt;      DWORD        dwData;
<br>&gt;&gt;      DWORD        dwTimeStamp;
<br>&gt;&gt;      DWORD        dwSequence;
<br>&gt;&gt; +#if(DIRECTINPUT_VERSION &gt;= 0x0800)
<br>&gt;&gt;      UINT_PTR        uAppData;
<br>&gt;&gt; +#endif
<br>&gt;
<br>&gt;As a rule we avoid that sort of thing in Wine (I know there are other
<br>&gt;such #ifs in that file but they should be removed). It makes no
<br>&gt;difference for binary compatibility, and it's very unlikely that an
<br>&gt;app would depend on it at the source level, so it's not worth the
<br>&gt;extra headaches.
<br>&gt;
<br>&gt;-- 
<br>&gt;Alexandre Julliard
<br>&gt;julliard@winehq.org
<br>&gt;
<br>&gt;

</fenix></julliard></fenix></body></html>