<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/11 Rico Schüller <span dir="ltr"><<a href="mailto:kgbricola@web.de" target="_blank">kgbricola@web.de</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 11.06.2013 22:08, Christian Costa wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Fixes bug 26598.<br>
---<br>
  dlls/d3dx9_36/effect.c       |    4 ++++<br>
  dlls/d3dx9_36/tests/effect.c |   17 +++++++++++++++++<br>
  2 files changed, 21 insertions(+)<br>
<br>
diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c<br>
index 1924c07..bab4560 100644<br>
--- a/dlls/d3dx9_36/effect.c<br>
+++ b/dlls/d3dx9_36/effect.c<br>
@@ -5792,6 +5792,10 @@ HRESULT WINAPI D3DXCreateEffectEx(struct IDirect3DDevice9 *device, const void *s<br>
<br>
      *effect = &object->ID3DXEffect_iface;<br>
<br>
+    /* Must be set to NULL if no compilation error */<br>
+    if (compilation_errors)<br>
+        *compilation_errors = NULL;<br>
+<br>
</blockquote></div>
No, this is wrong! Your test case doesn't cover all cases.<br>
<br>
</blockquote><div>Which cases? Ssource effects or possible parsing error of binary ones?<br></div><div>Anyway my intention was to fix only this particular case as we don't support source effect yet.<br>I'll use a different approach.<br>
</div><div><br></div></div></div></div>