Re: [Mesa-dev] [PATCH 2/3] egl/x11: store xcb_screen_t *screen instead of int screen

2016-11-21 Thread Emil Velikov
On 17 November 2016 at 13:59, Eric Engestrom wrote: > On Friday, 2016-11-11 16:31:16 +, Emil Velikov wrote: >> From: Emil Velikov >> >> Just fetch and store it once, rather than doing the >> xcb_setup_roots_iterator + get_xcb_screen

Re: [Mesa-dev] [PATCH 2/3] egl/x11: store xcb_screen_t *screen instead of int screen

2016-11-17 Thread Eric Engestrom
On Friday, 2016-11-11 16:31:16 +, Emil Velikov wrote: > From: Emil Velikov > > Just fetch and store it once, rather than doing the > xcb_setup_roots_iterator + get_xcb_screen dance five times. > > Signed-off-by: Emil Velikov > --- >

[Mesa-dev] [PATCH 2/3] egl/x11: store xcb_screen_t *screen instead of int screen

2016-11-11 Thread Emil Velikov
From: Emil Velikov Just fetch and store it once, rather than doing the xcb_setup_roots_iterator + get_xcb_screen dance five times. Signed-off-by: Emil Velikov --- These two patches are an example of the duplication we have within each