[Mesa-dev] [PATCH] mesa: add missing ASSERT_OUTSIDE_BEGIN_END() in _mesa_GetInternalformativ()

Ian Romanick idr at freedesktop.org
Wed Jan 16 15:45:25 PST 2013


On 01/16/2013 07:19 AM, Brian Paul wrote:

Ah, yes.  I always forget about that.  I can't wait until Eric finishes 
separating the dispatch tables so that we never have to do this again.

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

> ---
>   src/mesa/main/formatquery.c |    3 +++
>   1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/src/mesa/main/formatquery.c b/src/mesa/main/formatquery.c
> index f08ab66..bd895e8 100644
> --- a/src/mesa/main/formatquery.c
> +++ b/src/mesa/main/formatquery.c
> @@ -22,6 +22,7 @@
>    */
>
>   #include "mtypes.h"
> +#include "context.h"
>   #include "glformats.h"
>   #include "macros.h"
>   #include "mfeatures.h"
> @@ -37,6 +38,8 @@ _mesa_GetInternalformativ(GLenum target, GLenum internalformat, GLenum pname,
>      GLsizei count = 0;
>      GET_CURRENT_CONTEXT(ctx);
>
> +   ASSERT_OUTSIDE_BEGIN_END(ctx);
> +
>      if (!ctx->Extensions.ARB_internalformat_query) {
>         _mesa_error(ctx, GL_INVALID_OPERATION, "glGetInternalformativ");
>         return;
>



More information about the mesa-dev mailing list