wined3d: Add hardware cursor support.
Andrew Riedi
andrewriedi at gmail.com
Fri May 11 07:54:22 CDT 2007
You are correct, a simple is just as good:
+ if (This->hardwareCursor) DestroyCursor(This->hardwareCursor);
+ This->haveHardwareCursor = FALSE;
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:
if (somePtr) FreeThis(somePtr);
I tried with the new code above, and everything works just as well.
On 5/11/07, Stefan Dösinger < stefandoesinger at gmx.at> wrote:
>
> Am Freitag 11 Mai 2007 02:34 schrieb Andrew Riedi:
> > I made a test to figure out the exact Windows behavior, and Stefand
> helped
> > me out in telling me what to test. This code behaves how it should. I
> > will send the test in later once I have cleaned it up.
> >
> > changelog: wined3d: Add hardware cursor support.
> > + if (This->hardwareCursor) DestroyCursor(This->hardwareCursor);
> > + if (This->haveHardwareCursor) This->haveHardwareCursor = FALSE;
>
> Is there a specific reason to test if This->haveHardwareCursor is true
> before
> setting it to false? Or would a plain This->haveHardwareCursor = FALSE
> work
> too?
>
> Otherwise looked fine to me
>
>
>
>
>
--
Andrew Riedi
--
Andrew Riedi
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.winehq.org/pipermail/wine-patches/attachments/20070511/30b700cc/attachment.html
More information about the wine-patches
mailing list