[PATCH 3/2] Track screens' installed colormaps as screen privates.

Tiago Vignatti tiago.vignatti at nokia.com
Thu Apr 22 10:07:58 PDT 2010


On Thu, Apr 22, 2010 at 07:34:04AM +0200, ext Jamey Sharp wrote:
> Several DDXes allow each screen to have at most one (or in some cases,
> exactly one) installed colormap. These all use the same pattern: Declare
> a global-lifetime array of MAXSCREENS ColormapPtrs, and index it by
> screen number. This patch converts most of those to use screen privates
> instead.

Acked-by: Tiago Vignatti <tiago.vignatti at nokia.com>

(applied it in my tree containing "maxscreenless" work)


> mi/micmap.c has the same pattern, except there the miInstalledMaps array
> is used in the xfree86 DDX, and I guess it's part of the server's ABI.
> So for now I left it alone.

If we get cooked this and target to apply this before 1.9 merge window be
closed, so why bother? Even though, I removed this commentary from this patch
which seems unrelated.


         Tiago


More information about the xorg-devel mailing list