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.