tobert at gmail.com
Mon Jan 16 09:06:01 CST 2006
Thanks for the feedback everybody. I'll rewrite it along with a test
of some sort tonight.
On 1/16/06, Aric Cyr <aric.cyr at gmail.com> wrote:
> On 1/16/06, H. Verbeet <hverbeet at gmail.com> wrote:
> > If the D3DDEVICE_CREATION_PARAMETERS struct got zeroed out before
> > being passed to GetCreationParameters, DeviceType would now be 0 while
> > it was 1 (D3DDEVTYPE_HAL) before. I don't know if that's enough to
> > make things work.
> Yes good point. I forgot that his original patch also included the
> deletion of a few lines. That is the only reason I could see for his
> changes to actually have any effect as the pointer-assignment clearly
> does nothing.
> > And of course checking if pParameters is a valid pointer in the first
> > place is probably a good idea as well.
> I agree, definitely a NULL check should be added for pParameters.
> Aric Cyr <Aric.Cyr at gmail dot com> (http://acyr.net)
More information about the wine-devel