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]> -Emil _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
