Re: [Mesa-dev] [PATCH v2] egl: Fix missing clamping in eglSetDamageRegionKHR

2018-07-24 Thread Dylan Baker
Quoting Eric Engestrom (2018-07-24 07:55:03) > On Tuesday, 2018-07-24 18:15:54 +0530, Harish Krupo wrote: > > Hi Eric, > > > > Eric Engestrom writes: > > > > > On Sunday, 2018-07-08 12:53:00 +0530, Harish Krupo wrote: > > >> Clamp the x and y co-ordinates of the rectangles. > > >> > > >> v2: Cl

Re: [Mesa-dev] [PATCH v2] egl: Fix missing clamping in eglSetDamageRegionKHR

2018-07-24 Thread Eric Engestrom
On Tuesday, 2018-07-24 18:15:54 +0530, Harish Krupo wrote: > Hi Eric, > > Eric Engestrom writes: > > > On Sunday, 2018-07-08 12:53:00 +0530, Harish Krupo wrote: > >> Clamp the x and y co-ordinates of the rectangles. > >> > >> v2: Clamp width/height after converting to co-ordinates > >> (Ili

Re: [Mesa-dev] [PATCH v2] egl: Fix missing clamping in eglSetDamageRegionKHR

2018-07-24 Thread Harish Krupo
Hi Eric, Eric Engestrom writes: > On Sunday, 2018-07-08 12:53:00 +0530, Harish Krupo wrote: >> Clamp the x and y co-ordinates of the rectangles. >> >> v2: Clamp width/height after converting to co-ordinates >> (Ilia Merkin) >> >> Signed-off-by: Harish Krupo >> --- >> src/egl/main/eglapi.

Re: [Mesa-dev] [PATCH v2] egl: Fix missing clamping in eglSetDamageRegionKHR

2018-07-24 Thread Eric Engestrom
On Sunday, 2018-07-08 12:53:00 +0530, Harish Krupo wrote: > Clamp the x and y co-ordinates of the rectangles. > > v2: Clamp width/height after converting to co-ordinates > (Ilia Merkin) > > Signed-off-by: Harish Krupo > --- > src/egl/main/eglapi.c | 25 +++-- > 1 file ch

Re: [Mesa-dev] [PATCH v2] egl: Fix missing clamping in eglSetDamageRegionKHR

2018-07-24 Thread Tapani Pälli
Reviewed-by: Tapani Pälli On 07/08/2018 10:23 AM, Harish Krupo wrote: Clamp the x and y co-ordinates of the rectangles. v2: Clamp width/height after converting to co-ordinates (Ilia Merkin) Signed-off-by: Harish Krupo --- src/egl/main/eglapi.c | 25 +++-- 1 file c

Re: [Mesa-dev] [PATCH v2] egl: Fix missing clamping in eglSetDamageRegionKHR

2018-07-14 Thread Qiang Yu
Looks good for me. Reviewed-by: Qiang Yu Regards, Qiang On Thu, Jul 12, 2018 at 12:28 AM Harish Krupo wrote: > > > Harish Krupo writes: > > > Clamp the x and y co-ordinates of the rectangles. > > > > v2: Clamp width/height after converting to co-ordinates > > (Ilia Merkin) > > > > Signed-of

Re: [Mesa-dev] [PATCH v2] egl: Fix missing clamping in eglSetDamageRegionKHR

2018-07-11 Thread Harish Krupo
Harish Krupo writes: > Clamp the x and y co-ordinates of the rectangles. > > v2: Clamp width/height after converting to co-ordinates > (Ilia Merkin) > > Signed-off-by: Harish Krupo > --- > src/egl/main/eglapi.c | 25 +++-- > 1 file changed, 11 insertions(+), 14 deletion

[Mesa-dev] [PATCH v2] egl: Fix missing clamping in eglSetDamageRegionKHR

2018-07-08 Thread Harish Krupo
Clamp the x and y co-ordinates of the rectangles. v2: Clamp width/height after converting to co-ordinates (Ilia Merkin) Signed-off-by: Harish Krupo --- src/egl/main/eglapi.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/src/egl/main/eglapi.c