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