[PATCH xserver] Autoconfig: Fix the loop adding dectected drivers to the list of screens

Keith Packard keithp at keithp.com
Sat Jul 30 22:39:41 UTC 2016


Matthieu Herrb <matthieu at herrb.eu> writes:

> @@ -389,8 +389,7 @@ autoConfigDevice(GDevPtr preconf_device)
>  
>              /* for each other driver found, copy the first screen, insert it
>               * into the list of screens and set the driver */
> -            i = 0;
> -            while (i++ < num_matches) {
> +	    for (i = 0; i < num_matches; i++) {

Looks like the old loop started copying at 1, and just went one screen
too far. Should this be:

	    for (i = 1; i < num_matches; i++) {

-- 
-keith
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <https://lists.x.org/archives/xorg-devel/attachments/20160730/ff24ec1e/attachment.sig>


More information about the xorg-devel mailing list