[Mesa-dev] [PATCH] mesa/main: missing return in a non-void function

Emil Velikov emil.l.velikov at gmail.com
Thu Aug 25 15:19:03 UTC 2016


On 25 August 2016 at 13:24, Francesco Ansanelli <francians at gmail.com> wrote:
> Signed-off-by: Francesco Ansanelli <francians at gmail.com>
> ---
>  src/mesa/main/program_resource.c |    1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/main/program_resource.c b/src/mesa/main/program_resource.c
> index 6ddbdad..51dbe95 100644
> --- a/src/mesa/main/program_resource.c
> +++ b/src/mesa/main/program_resource.c
> @@ -105,6 +105,7 @@ stage_from_program_interface(GLenum programInterface)
>        return MESA_SHADER_COMPUTE;
>     default:
>        assert(!"unexpected programInterface value");
> +      return -1;
This should be unreachable() since programInterface is validated
further up the stack by is_subroutine_uniform_program_interface().

-Emil


More information about the mesa-dev mailing list