On Tue, 2017-03-21 at 15:07 +0000, Emil Velikov wrote: > > On 9 March 2017 at 08:34, Michel Dänzer <[email protected]> wrote: > > > > From: Michel Dänzer <[email protected]> > > > > 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 <[email protected]> > > Patch looks spot on. Fwiw > Reviewed-by: Emil Velikov <[email protected]>
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 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
