svga: use switch statement in svga_shader_type()

Safer in case the PIPE_SHADER_x tokens get renumbered (as Marek
wanted to do).

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
This commit is contained in:
Brian Paul 2013-06-28 08:08:38 -06:00
parent 24b05ff158
commit 74e8a7d1dd
1 changed files with 10 additions and 5 deletions

View File

@ -46,13 +46,18 @@
/**
* Convert from PIPE_SHADER_* to SVGA3D_SHADERTYPE_*
*/
static int
static unsigned
svga_shader_type(unsigned shader)
{
assert(PIPE_SHADER_VERTEX + 1 == SVGA3D_SHADERTYPE_VS);
assert(PIPE_SHADER_FRAGMENT + 1 == SVGA3D_SHADERTYPE_PS);
assert(shader <= PIPE_SHADER_FRAGMENT);
return shader + 1;
switch (shader) {
case PIPE_SHADER_VERTEX:
return SVGA3D_SHADERTYPE_VS;
case PIPE_SHADER_FRAGMENT:
return SVGA3D_SHADERTYPE_PS;
default:
assert(!"Unexpected shader type");
return SVGA3D_SHADERTYPE_VS;
}
}