Re: [PATCH libdrm v2 04/10] xf86drm: Allocate drmDevicePtr's on stack

2018-06-29 Thread Robert Foss
LGTM On 2018-06-29 17:20, Emil Velikov wrote: From: Emil Velikov Currently we dynamically allocate 16 pointers and reallocate more as needed. Instead, allocate the maximum number (256) on stack - the number is small enough and is unlikely to change in the foreseeable future. This allows us t

[PATCH libdrm v2 04/10] xf86drm: Allocate drmDevicePtr's on stack

2018-06-29 Thread Emil Velikov
From: Emil Velikov Currently we dynamically allocate 16 pointers and reallocate more as needed. Instead, allocate the maximum number (256) on stack - the number is small enough and is unlikely to change in the foreseeable future. This allows us to simplify the error handling and even shed a few