Dave Airlie <airl...@gmail.com> 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 <kei...@keithp.com>

-- 
keith.pack...@intel.com

Attachment: pgp91fobDpqYv.pgp
Description: PGP signature

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to