Brian Paul <[email protected]> writes:

> With the explit NUM_TEXTURE_TARGETS array size, the assertion that
> Elements(targets) == NUM_TEXTURE_TARGETS would pass even if elements
> were missing.
> ---
>  src/mesa/main/shared.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/mesa/main/shared.c b/src/mesa/main/shared.c
> index fab2995..2d2f7bd 100644
> --- a/src/mesa/main/shared.c
> +++ b/src/mesa/main/shared.c
> @@ -89,7 +89,7 @@ _mesa_alloc_shared_state(struct gl_context *ctx)
>     /* Create default texture objects */
>     for (i = 0; i < NUM_TEXTURE_TARGETS; i++) {
>        /* NOTE: the order of these enums matches the TEXTURE_x_INDEX values */
> -      static const GLenum targets[NUM_TEXTURE_TARGETS] = {
> +      static const GLenum targets[] = {
>           GL_TEXTURE_BUFFER,
>           GL_TEXTURE_2D_ARRAY_EXT,
>           GL_TEXTURE_1D_ARRAY_EXT,

Reviewed-by: Eric Anholt <[email protected]>

Attachment: pgpsqZudClxt2.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to