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 <mar...@gmail.com> wrote:
> Reviewed-by: Marek Olšák <marek.ol...@amd.com>
>
> Marek
>
> On Thu, May 4, 2017 at 5:13 PM, Kenneth Graunke <kenn...@whitecape.org> 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.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.12.2
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
