From 8c5a938171d0566b087fdfb5c422464aa005e8ed Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Thu, 4 May 2017 01:08:25 -0700 Subject: [PATCH] mesa: Simplify _mesa_primitive_restart_index(). MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit We can use a simple shift equation rather than a switch statement. Reviewed-by: Rafael Antognolli Reviewed-by: Marek Olšák --- 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/src/mesa/main/varray.c @@ -1959,16 +1959,8 @@ _mesa_primitive_restart_index(const struct gl_context *ctx, * is used." */ if (ctx->Array.PrimitiveRestartFixedIndex) { - switch (index_size) { - case 1: - return 0xff; - case 2: - return 0xffff; - case 4: - return 0xffffffff; - default: - assert(!"_mesa_primitive_restart_index: Invalid index size."); - } + /* 1 -> 0xff, 2 -> 0xffff, 4 -> 0xffffffff */ + return 0xffffffffu >> 8 * (4 - index_size); } return ctx->Array.RestartIndex; -- 2.30.2