[try4 3/3] gdi32: added PATH_PolyDraw

Misha Koshelev mk144210 at bcm.edu
Wed Jul 4 14:58:01 CDT 2007


> +            else if((types[i] & ~PT_CLOSEFIGURE) == PT_LINETO){
> +                PATH_LineTo(dc, pts[i].x, pts[i].y);
> +            }

> +                if(!((i + 2 < cbPoints) && (types[i + 1] == PT_BEZIERTO)
> +                    && ((types[i + 2] & ~PT_CLOSEFIGURE) == PT_BEZIERTO))){
> +                    goto err;
> +                }

> +            else{
> +                goto err;
> +            }

On a more stylistic note, any reason for all these single line if statements that
have their contents enclosed in {}'s? (i.e., instead of:

if (a) {
   call_b();
}

why not just:

if (a)
   call_b();

If anything, this adds one unnecessary line with the single } after the statement itself, and
seems unnecessary to me.

Misha



More information about the wine-devel mailing list