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

Vignatti Tiago (Nokia-D/Helsinki) tiago.vignatti at nokia.com
Fri Apr 23 06:31:13 PDT 2010


Hey Jamey,

On Fri, Apr 23, 2010 at 07:02:54AM +0200, ext Jamey Sharp wrote:
> On Thu, Apr 22, 2010 at 10:42 AM, Vignatti Tiago (Nokia-D/Helsinki)
> >>
> >> 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.
> 
> I'm sorry, I didn't understand that. Do you mean it's OK to make ABI
> changes for 1.9 so long as we get a bunch of them done at once and
> remember to bump the ABI version?

Yeah, that what I meant... (see bellow)
 
> Excellent. The bulk of the patch makes perfect sense to me. There are
> a couple things I'd like to understand still though:
> 
> Since this adds and removes globals marked _X_EXPORT, it is an ABI
> change, right? I'm still trying to work out what the rules are.

probably yeah.


Please disregard any comment regarding ABI/API being break here. I didn't know
what I was talking previously, but I got already a feedback from some guys in
IRC.

 
> Does micmapScrPrivateKeyIndex need to be exported? AFAICT nothing
> should ever refer to it except the initializer for
> micmapScrPrivateKey, so it can be declared static, right?

you are right. I just amend it. 


Thanks,

                            Tiago


More information about the xorg-devel mailing list