<span class="gmail_quote"></span>You are correct, a simple is just as good:<br> + if (This->hardwareCursor) DestroyCursor(This->hardwareCursor);<br> + This->haveHardwareCursor = FALSE;<br><br>Think I must have had this patch on my computer for too long, and at the time I wrote that, I must have been working with stuff like:
<br>if (somePtr) FreeThis(somePtr);<br><br>I tried with the new code above, and everything works just as well.<div><span class="e" id="q_1127b32023dea1df_1"><br><br><div><span class="gmail_quote">On 5/11/07, <b class="gmail_sendername">
Stefan D�singer</b> <<a href="mailto:stefandoesinger@gmx.at" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
stefandoesinger@gmx.at</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Am Freitag 11 Mai 2007 02:34 schrieb Andrew Riedi:
<br>> I made a test to figure out the exact Windows behavior, and Stefand helped<br>> me out in telling me what to test. This code behaves how it should. I<br>> will send the test in later once I have cleaned it up.
<br>><br>> changelog: wined3d: Add hardware cursor support.<br>> + if (This->hardwareCursor) DestroyCursor(This->hardwareCursor);<br>> + if (This->haveHardwareCursor) This->haveHardwareCursor = FALSE;
<br>Is there a specific reason to test if This->haveHardwareCursor is true before<br>setting it to false? Or would a plain This->haveHardwareCursor = FALSE work<br>too?<br><br>Otherwise looked fine to me<br><br><br>
<br><br></blockquote></div><br><br clear="all"><br></span></div>-- <br><span class="sg">Andrew Riedi
</span><br clear="all"><br>-- <br>Andrew Riedi