Re: [ddraw] fix bug 3487: don't crash if pDDSD is NULL

[Raphael]

On Thursday 06 October 2005 09:38, Vitaliy Margolen wrote:
> Thursday, October 6, 2005, 1:31:09 AM, Raphael wrote:
> > Hi,
> >  Changelog:
> >   - fix 3487: don't crash if pDDSD is NULL on DirectDrawSurface::Lock
> > Regards,
> > Raphael
> > +    if (NULL == lpddsd) {
> > +      DPRINTF("(null)\n");
> >      } else {
>
> Please don't use DPFINTF

Why ? I use DPRINTF as all used by all ddraw-utils traces
> > +      if (lpddsd->>dwSize >= sizeof(DDSURFACEDESC2)) {
> > +        DDRAW_dump_members(lpddsd->dwFlags, lpddsd, members_caps2, 1);
> > +      } else {
>
> And follow the same indentation style (4 spaces):

emacs is my friend :)
> Vitaliy Margolen

Regards,
Raphael