Re: [Mesa-dev] [PATCH mesa 1/3] egl: deduplicate swap interval clamping logic

2017-08-01 Thread Tapani Pälli
lgtm Reviewed-by: Tapani Pälli On 08/01/2017 03:55 PM, Eric Engestrom wrote: Signed-off-by: Eric Engestrom Reviewed-by: Daniel Stone --- src/egl/drivers/dri2/platform_wayland.c | 14 +- src/egl/drivers/dri2/platform_x11.c | 9 + src/egl/main/eglapi.c

Re: [Mesa-dev] [PATCH mesa 1/3] egl: deduplicate swap interval clamping logic

2017-08-01 Thread Emil Velikov
On 1 August 2017 at 13:55, Eric Engestrom wrote: > Signed-off-by: Eric Engestrom > Reviewed-by: Daniel Stone If we only had implementation swap_interval implementation across the board, we could have simplified things ever further. Regardless, nice cleanup. For the lot Reviewed-by: Emil Velikov

[Mesa-dev] [PATCH mesa 1/3] egl: deduplicate swap interval clamping logic

2017-08-01 Thread Eric Engestrom
Signed-off-by: Eric Engestrom Reviewed-by: Daniel Stone --- src/egl/drivers/dri2/platform_wayland.c | 14 +- src/egl/drivers/dri2/platform_x11.c | 9 + src/egl/main/eglapi.c | 12 +++- src/egl/main/eglsurface.c | 19 +--