[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: [PATCH 3/3] gdiplus:
From:       Nikolay Sivov <nsivov () codeweavers ! com>
Date:       2010-03-27 22:08:27
Message-ID: 4BAE81DB.7020804 () codeweavers ! com
[Download RAW message or body]

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.



[prev in list] [next in list] [prev in thread] [next in thread]