Re: [GDI+: 3/10] graphics implementation

2007-06-12 Thread Alexandre Julliard
Detlef Riekenberg <[EMAIL PROTECTED]> writes: > Have you checked, that the native dll does the same? > (WINEDEBUG="+snoop,+relay") There's no reason to look at what calls native makes, that's an internal implementation detail that we don't want to know about. -- Alexandre Julliard [EMAIL PROTEC

Re: [GDI+: 3/10] graphics implementation

2007-06-12 Thread Detlef Riekenberg
On Mo, 2007-06-11 at 11:54 -0700, Evan Stade wrote: > + > +*graphics = (GpGraphics*) HeapAlloc(GetProcessHeap(), > HEAP_ZERO_MEMORY, > +sizeof(GpGraphics)); Have you checked, that the native dll does the same? (WINEDEBUG="+snoop,+relay") Your implementation of GdipCreatePen1 is usi

Re: [GDI+: 3/10] graphics implementation

2007-06-12 Thread James Hawkins
On 6/12/07, Detlef Riekenberg <[EMAIL PROTECTED]> wrote: On Mo, 2007-06-11 at 11:54 -0700, Evan Stade wrote: > +GpStatus WINGDIPAPI GdipCreateFromHDC(HDC hdc, GpGraphics **graphics) > +{ > +if(hdc == NULL) > +return OutOfMemory; > + > +if(graphics == NULL) > +return Invali

Re: [GDI+: 3/10] graphics implementation

2007-06-12 Thread Detlef Riekenberg
On Mo, 2007-06-11 at 11:54 -0700, Evan Stade wrote: > +GpStatus WINGDIPAPI GdipCreateFromHDC(HDC hdc, GpGraphics **graphics) > +{ > +if(hdc == NULL) > +return OutOfMemory; > + > +if(graphics == NULL) > +return InvalidParameter; > + > +*graphics = (GpGraphics*) HeapAlloc(