On 13/11/17 06:20 PM, John Lumby wrote:
>
> --- xserver/hw/xfree86/modes/xf86EdidModes.c.orig 2017-09-24
> 18:29:54.582068856 -0400
> +++ xserver/hw/xfree86/modes/xf86EdidModes.c 2017-11-13 12:06:11.843310650
> -0500
> @@ -1203,7 +1203,10 @@ xf86EdidMonitorSet(int scrnIndex, MonPtr
> Monitor->Modes = Modes;
> }
>
> - xf86PruneDuplicateModes(Monitor->Modes);
> + /* xf86PruneDuplicateModes may delete any item in the list including
> the first,
> + * but always returns pointer to new start of list
> + */
> + Monitor->Modes = xf86PruneDuplicateModes(Monitor->Modes);
>
> /* Update pointer to last mode */
> for (Mode = Monitor->Modes; Mode && Mode->next; Mode = Mode->next) {}
The code change looks good, good catch.
Please take a look at
https://www.x.org/wiki/Development/Documentation/SubmittingPatches/ for
how to properly create a patch and send it for review.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel
