Dave Airlie <[email protected]> writes:

> +
> +    for (layout = xf86ConfigLayout.screens; layout->screen != NULL;
> +         layout++) {
> +        xf86GPUScreens[i]->confScreen = layout->screen;
> +        break;
> +    }

This loop is insane.

Otherwise, this is all

Reviewed-by: Keith Packard <[email protected]>

-- 
[email protected]

Attachment: pgp91fobDpqYv.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to