[Mesa-dev] [PATCH] mesa: Add missing break statement in _mesa_choose_tex_format.

Brian Paul brianp at vmware.com
Tue May 28 07:14:27 PDT 2013


On 05/27/2013 07:04 PM, Vinson Lee wrote:
> Fixes "Missing break in switch" defect reported by Coverity.
>
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>   src/mesa/main/texformat.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/main/texformat.c b/src/mesa/main/texformat.c
> index fda5d74..0ae79ea 100644
> --- a/src/mesa/main/texformat.c
> +++ b/src/mesa/main/texformat.c
> @@ -218,6 +218,7 @@ _mesa_choose_tex_format(struct gl_context *ctx, GLenum target,
>         RETURN_IF_SUPPORTED(MESA_FORMAT_Z16);
>         RETURN_IF_SUPPORTED(MESA_FORMAT_X8_Z24);
>         RETURN_IF_SUPPORTED(MESA_FORMAT_S8_Z24);
> +      break;
>
>    case GL_COMPRESSED_ALPHA_ARB:
>       RETURN_IF_SUPPORTED(MESA_FORMAT_A8);
>

Reviewed-by: Brian Paul <brianp at vmware.com>


More information about the mesa-dev mailing list