Re: [PATCH 3/3] gdiplus: Implement GdipSetPathGradientSurroundColorsWithCount and GdipGetPathGradientSurroundColorCount

[Nikolay Sivov]

On 3/28/2010 00:55, Justin Chevrier wrote:
> GpStatus WINGDIPAPI GdipGetPathGradientWrapMode(GpPathGradient *brush,
> @@ -1544,18 +1546,16 @@ GpStatus WINGDIPAPI GdipSetPathGradientSigmaBlend(GpPathGradient *grad,
>   GpStatus WINGDIPAPI GdipSetPathGradientSurroundColorsWithCount(GpPathGradient
>       *grad, GDIPCONST ARGB *argb, INT *count)
>   {
> -
>
> static int calls;
> -
> TRACE("(%p,%p,%p)\n", grad, argb, count);
>
> if(!grad || !argb || !count || (*count<= 0) ||
>     (*count>  grad->pathdata.Count))
>     return InvalidParameter;
>
> -
>
> if(!(calls++))
> -
>    FIXME("not implemented\n");
> - grad->surroundcolorcount = *count;
> - grad->surroundcolors = argb;
> - return NotImplemented;
> - return Ok;
> }

You should make a copy of colors array here. Wrapper method will free 
array after a call.