[Mesa-dev] [PATCH 1/3] mesa: Simplify _mesa_primitive_restart_index().

We can use a simple shift equation rather than a switch statement. --- src/mesa/main/varray.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c index eda86ec6a82..9497090e88a 100644 --- a/src/mesa/main/varray.c +++ b/s

Re: [Mesa-dev] [PATCH 1/3] mesa: Simplify _mesa_primitive_restart_index().

Reviewed-by: Rafael Antognolli On Thu, May 04, 2017 at 08:13:05AM -0700, Kenneth Graunke wrote: > We can use a simple shift equation rather than a switch statement. > --- > src/mesa/main/varray.c | 12 ++-- > 1 file changed, 2 insertions(+), 10 deletions(-) > > diff --git a/src/mesa/mai

Re: [Mesa-dev] [PATCH 1/3] mesa: Simplify _mesa_primitive_restart_index().

Reviewed-by: Marek Olšák Marek On Thu, May 4, 2017 at 5:13 PM, Kenneth Graunke wrote: > We can use a simple shift equation rather than a switch statement. > --- > src/mesa/main/varray.c | 12 ++-- > 1 file changed, 2 insertions(+), 10 deletions(-) > > diff --git a/src/mesa/main/varray.

Re: [Mesa-dev] [PATCH 1/3] mesa: Simplify _mesa_primitive_restart_index().

Since the function is so small, perhaps it would make sense to move it into a header file. Marek On Fri, May 5, 2017 at 7:06 PM, Marek Olšák wrote: > Reviewed-by: Marek Olšák > > Marek > > On Thu, May 4, 2017 at 5:13 PM, Kenneth Graunke wrote: >> We can use a simple shift equation rather than