Re: [PATCH xserver 1/3] res: Simplify QueryClientPixmapBytes

2016-02-03 Thread Keith Packard
Adam Jackson writes: > I suspect this code predates the common resource hooks for computing > sizes. It's ugly in any case since the Resource extension shouldn't > need to know which extensions can take a reference on pixmaps. Instead, > let's just walk every resource for the

Re: [PATCH xserver 1/3] res: Simplify QueryClientPixmapBytes

2016-02-03 Thread Keith Packard
Adam Jackson writes: > I don't really understand your objection about pixmaps not in the > resdb, because ResQueryClientPixmapBytes is _already_ not accounting > for those, and it's not clear to me how it even could. Sorry, that code has gotten a lot more sophisticated since I

Re: [PATCH xserver 1/3] res: Simplify QueryClientPixmapBytes

2016-02-03 Thread Adam Jackson
On Wed, 2016-02-03 at 21:34 +1100, Keith Packard wrote: > Adam Jackson writes: > > > I suspect this code predates the common resource hooks for computing > > sizes.  It's ugly in any case since the Resource extension shouldn't > > need to know which extensions can take a