On Thu, Nov 2, 2017 at 4:57 AM, Andres Rodriguez <andre...@gmail.com> wrote:
> EXT_semaphore and EXT_semaphore_fd define no pnames. Therefore there
> isn't much to do besides determining the correct error code.
>
> Signed-off-by: Andres Rodriguez <andre...@gmail.com>
> ---
> src/mesa/main/externalobjects.c | 23 +++++++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
> diff --git a/src/mesa/main/externalobjects.c b/src/mesa/main/externalobjects.c
> index 30f5248..93a92e9 100644
> --- a/src/mesa/main/externalobjects.c
> +++ b/src/mesa/main/externalobjects.c
> @@ -696,12 +696,33 @@ _mesa_IsSemaphoreEXT(GLuint semaphore)
> return obj ? GL_TRUE : GL_FALSE;
> }
>
> +/**
> + * Helper that outputs the correct error status for parameter
> + * calls where no pnames are defined
> + */
> +static void
> +semaphore_parameter_stub(const char* func, GLenum pname)
> +{
> + GET_CURRENT_CONTEXT(ctx);
> +
> + if (!ctx->Extensions.EXT_semaphore) {
> + _mesa_error(ctx, GL_INVALID_OPERATION, "%s(unsupported)", func);
> + return;
> + }
> +
> + /* EXT_semaphore and EXT_semaphore_fd define no parameters */
> + _mesa_error(ctx, GL_INVALID_ENUM, "%s(pname=0x%x)", func, pname);
> + return;
Useless return;
Marek
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
