Re: resend : gdi32:WidenPath partial implementation

[Alexandre Julliard]

Laurent Vromman <laurent(a)vromman.org> writes:
> +
> +    dc = DC_GetDCPtr( hdc );

You shouldn't need that, the caller has done it already. Simply pass a
DC* to this function.
> +    if(pPath->state != PATH_Closed) {
> +       ERR("Path Closed\n");
> +       return FALSE;
> +    }
> +
> +    hPen = SelectObject(hdc, GetStockObject(DC_PEN));

You shouldn't select a new pen into the DC, just use the current one.
> +            case PT_BEZIERTO:
> +                // should never happen because of the FlattenPath call

Please avoid C++ comments.
> +            default:
> +                ERR("Got path flag %d\n", (INT)pPath->pFlags[i]);

Please avoid casts like this, use the correct printf format instead.

-- 
Alexandre Julliard
julliard(a)winehq.org