Re: [Mesa-dev] [PATCH] mesa/st: only compare the current scissor

2014-03-07 Thread Ilia Mirkin
On Fri, Mar 7, 2014 at 8:01 AM, Ian Romanick wrote: > On 03/06/2014 05:38 AM, Ilia Mirkin wrote: >> sizeof(scissor) returns the size of the full array rather than a single >> element. Fix it to consider just the one element. >> >> Fixes: 0705fa35cdaf15ec969c28dc85e88b8be1149a3b >> Signed-off-by: I

Re: [Mesa-dev] [PATCH] mesa/st: only compare the current scissor

2014-03-07 Thread Ian Romanick
On 03/06/2014 05:38 AM, Ilia Mirkin wrote: > sizeof(scissor) returns the size of the full array rather than a single > element. Fix it to consider just the one element. > > Fixes: 0705fa35cdaf15ec969c28dc85e88b8be1149a3b > Signed-off-by: Ilia Mirkin Yeah, this looks right. Reviewed-by: Ian Roma

[Mesa-dev] [PATCH] mesa/st: only compare the current scissor

2014-03-05 Thread Ilia Mirkin
sizeof(scissor) returns the size of the full array rather than a single element. Fix it to consider just the one element. Fixes: 0705fa35cdaf15ec969c28dc85e88b8be1149a3b Signed-off-by: Ilia Mirkin --- Noticed this when playing with valgrind for unrelated reasons. Seems fairly obvious. src/mesa