[PATCH xserver v2] edid: Prune duplicates after adding modes from DDC

[Adam Jackson]

On Tue, 2017-03-21 at 15:07 +0000, Emil Velikov wrote:
> > On 9 March 2017 at 08:34, Michel Dänzer <michel at daenzer.net> wrote:
> > > > From: Michel Dänzer <michel.daenzer at amd.com>
> > 
> > Multiple calls to xf86EdidMonitorSet (which can be triggered e.g. by
> > running xrandr) would potentially keep adding the same modes, causing
> > the Monitor->Modes list to keep growing larger and using up more memory.
> > 
> > Fix this by calling xf86PruneDuplicateModes after adding the modes
> > returned by xf86DDCGetModes. This makes sure there's only one instance
> > of each unique mode in the list.
> > 
> > v2:
> > * Replace semicolon with {} for empty for loop (Emil Velikov)
> > * Slightly tweak commit log to avoid minor inaccuracy about what
> >   xf86PruneDuplicateModes does
> > 
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99521
> > Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
> 
> Patch looks spot on. Fwiw
> Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>

remote: I: patch #143158 updated using rev fdc79fe72bc0b97776df2c3a664076c60e08a87c.
remote: I: 1 patch(es) updated to state Accepted.
To ssh://git.freedesktop.org/git/xorg/xserver
   4e76c27..fdc79fe  master -> master

- ajax