Hello,<br><br>some comments about this patch<br>+    if (hr != S_OK) return FALSE;<br>+    else return TRUE;<br><br>could be simplified as<br>return SUCCEEDED(hr);<br><br>+    trace(&quot;HRESULT when creating instance (%x)\n&quot;, hr);<br>
Since tests are assumed to pass, we dont care about debug informations.  So, traces are useless in tests. [It is my opinion. Better developpers could give an opposite opinion]<br><br>+    todo_wine ok(*(ULONG *)pDirectMusicPort != 0, &quot;IDirectMusicPort not set\n&quot;);<br>
<br>Why is this cast useful?<br><br><br>A+<br>David<br><br><div class="gmail_quote">2010/8/9 Austin Lund <span dir="ltr">&lt;<a href="mailto:austin.lund@gmail.com">austin.lund@gmail.com</a>&gt;</span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
<br><br>
<br></blockquote></div><br>