kannonier8/blender
1
0
Fork 0
forked from bartvdbraak/blender
Code Pull Requests Activity

Gawain: replace switch with lookup table

Browse Source 
This function is not performance critical, but I prefer the branch-free code and no hack needed to appease gcc.

Follow-up to recent 23035cf46fb4dd6a0bf7e688b0f15128030c77d1 and f637145450010d14660fcb029d41560a138eae14.
This commit is contained in:
Mike Erwin 2017-05-22 16:43:33 -04:00
parent 845732652f
commit 6cc293a6d9
1 changed files with 6 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
intern/gawain/src/element.c
View File

@ -28,19 +28,12 @@ static GLenum convert_index_type_to_gl(IndexType type)
unsigned ElementList_size(const ElementList* elem)
{
#if TRACK_INDEX_RANGE
switch (elem->index_type)
{
case INDEX_U8: return elem->index_ct * sizeof(GLubyte);
case INDEX_U16: return elem->index_ct * sizeof(GLushort);
case INDEX_U32: return elem->index_ct * sizeof(GLuint);
default:
#if TRUST_NO_ONE
assert(false);
#endif
return 0;
}
static const unsigned table[] = {
[INDEX_U8] = sizeof(GLubyte), // GL has this, Vulkan does not
[INDEX_U16] = sizeof(GLushort),
[INDEX_U32] = sizeof(GLuint)
};
return elem->index_ct * table[elem->index_type];
#else
return elem->index_ct * sizeof(GLuint);
#endif