[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: d3dx: A few last questions...
From:       "Luis_C._Busquets_Pérez"
Date:       2008-01-29 20:52:01
Message-ID: 479F91F1.4010603 () ilidium ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


I have submitted a patch for this function and it is easily convertable 
to the 13 different functions.

Anyway, propose another patch for it and let it be implemented.


tony.wasserka@freenet.de escribió:
> > The function D3DXCheckVersion must not be forwarded from each d3dx9_xx file to \
> > the 36 because D3DX_SDK_VERSION changes for each file.  
> 
> Thanks for your comment, though I don't know if this is needed:
> Comment from the d3dx9core.h of the DX SDK:
> 
> 
> > ///////////////////////////////////////////////////////////////////////////
> > // D3DX_SDK_VERSION:
> > // -----------------
> > // This identifier is passed to D3DXCheckVersion in order to ensure that an
> > // application was built against the correct header files and lib files. 
> > // This number is incremented whenever a header (or other) change would 
> > // require applications to be rebuilt. If the version doesn't match, 
> > // D3DXCheckVersion will return FALSE. (The number itself has no meaning.)
> > ///////////////////////////////////////////////////////////////////////////
> > 
> 
> So this function is only needed to ensure that no header file changed between the \
> last build of an application. As this won't be the case in our wine implementation \
> we can let all dlls use the same function.
> Also, the D3DX_SDK_VERSION is declared in d3dx9core.h, which means we can only have
> one definition of it at a time which even prevents us from caring about this \
> function. By the way, I think this function is just a leftover from the times, when \
> microsoft released updates of their D3DX library without renaming the dlls, were \
> using a more actual dll could really lead to conflicts.
> 
> 
> 
> 
> 
> Unbegrenzter Speicher, Top-Spamschutz, 120 SMS und eigene E-MailDomain inkl.
> http://office.freenet.de/dienste/emailoffice/produktuebersicht/power/mail/index.html
>  
> 


[Attachment #5 (text/html)]

<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
I have submitted a patch for this function and it is easily convertable
to the 13 different functions.<br>
<br>
Anyway, propose another patch for it and let it be implemented.<br>
<br>
<br>
<a class="moz-txt-link-abbreviated" \
href="mailto:tony.wasserka@freenet.de">tony.wasserka@freenet.de</a> escribi&oacute;: \
<blockquote cite="mid:E1JJvQy-00055p-KL@www13.emo.freenet-rz.de"  type="cite">
  <blockquote type="cite">
    <pre wrap="">The function D3DXCheckVersion must not be forwarded from each \
d3dx9_xx file to the 36 because D3DX_SDK_VERSION changes for each file.   </pre>
  </blockquote>
  <pre wrap=""><!---->
Thanks for your comment, though I don't know if this is needed:
Comment from the d3dx9core.h of the DX SDK:

  </pre>
  <blockquote type="cite">
    <pre wrap="">///////////////////////////////////////////////////////////////////////////
 // D3DX_SDK_VERSION:
// -----------------
// This identifier is passed to D3DXCheckVersion in order to ensure that an
// application was built against the correct header files and lib files. 
// This number is incremented whenever a header (or other) change would 
// require applications to be rebuilt. If the version doesn't match, 
// D3DXCheckVersion will return FALSE. (The number itself has no meaning.)
///////////////////////////////////////////////////////////////////////////
    </pre>
  </blockquote>
  <pre wrap=""><!---->
So this function is only needed to ensure that no header file changed between the \
last build of an application. As this won't be the case in our wine implementation we \
can let all dlls use the same function.
Also, the D3DX_SDK_VERSION is declared in d3dx9core.h, which means we can only have
one definition of it at a time which even prevents us from caring about this \
function. By the way, I think this function is just a leftover from the times, when \
microsoft released updates of their D3DX library without renaming the dlls, were \
using a more actual dll could really lead to conflicts.





Unbegrenzter Speicher, Top-Spamschutz, 120 SMS und eigene E-MailDomain inkl.
<a class="moz-txt-link-freetext" \
href="http://office.freenet.de/dienste/emailoffice/produktuebersicht/power/mail/index. \
html">http://office.freenet.de/dienste/emailoffice/produktuebersicht/power/mail/index.html</a>


  </pre>
</blockquote>
<br>
</body>
</html>





[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic