patch for Bug 103722

Michel Dänzer michel at daenzer.net
Wed Nov 15 17:56:53 UTC 2017


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


More information about the xorg-devel mailing list