Re: [PATCHv2] Remove static MAXSCREENS limit from Xext/shm.c.

2009-10-28 Thread Keith Packard
Excerpts from Jamey Sharp's message of Fri Oct 02 14:10:22 -0700 2009: I'll fix these and the lack of a CloseScreen cleanup hook and send another patch. Thanks! I haven't seen another version of this patch float by; did I miss it? -- keith.pack...@intel.com signature.asc Description: PGP

Re: [PATCHv2] Remove static MAXSCREENS limit from Xext/shm.c.

2009-10-02 Thread Eric Anholt
On Thu, 2009-10-01 at 10:47 -0700, Jamey Sharp wrote: --- I didn't test my previous patch right. Sorry. This version doesn't seem to crash the server at startup. :-) Review would still be greatly appreciated. Thanks for jumping in! Comments inline. Xext/shm.c | 67

Re: [PATCHv2] Remove static MAXSCREENS limit from Xext/shm.c.

2009-10-02 Thread Jamey Sharp
Thanks for review, Eric! On Thu, Oct 01, 2009 at 01:53:15PM -0700, Eric Anholt wrote: On Thu, 2009-10-01 at 10:47 -0700, Jamey Sharp wrote: +drawables = xcalloc(screenInfo.numScreens, sizeof(DrawablePtr)); Seems like this drawable pointer should be part of the screen private. That

[PATCHv2] Remove static MAXSCREENS limit from Xext/shm.c.

2009-10-01 Thread Jamey Sharp
--- I didn't test my previous patch right. Sorry. This version doesn't seem to crash the server at startup. :-) Review would still be greatly appreciated. Xext/shm.c | 67 +++ 1 files changed, 49 insertions(+), 18 deletions(-) diff