[Mesa-dev] [PATCH] mesa/tests: add KHR_debug GLES glGetPointervKHR entry points

[Matt Turner]

On Thu, Dec 3, 2015 at 2:05 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> Should have been part of commit f53f9eb8d49 "glapi: add GetPointervKHR
> to the ES dispatch".
>
> Note: as the core symbol is present in GLES 1.1 we cannot (should not)
> include the KHR one in the es11 table. Add the symbol, commented out,
> with description for posterity.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93235
> Fixes: f53f9eb8d49 "glapi: add GetPointervKHR to the ES dispatch".
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
>  src/mesa/main/tests/dispatch_sanity.cpp | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/mesa/main/tests/dispatch_sanity.cpp b/src/mesa/main/tests/dispatch_sanity.cpp
> index 97f81f9..687c8f3 100644
> --- a/src/mesa/main/tests/dispatch_sanity.cpp
> +++ b/src/mesa/main/tests/dispatch_sanity.cpp
> @@ -2049,6 +2049,8 @@ const struct function gles11_functions_possible[] = {
>     { "glGetDebugMessageLogKHR", 11, -1 },
>     { "glGetObjectLabelKHR", 11, -1 },
>     { "glGetObjectPtrLabelKHR", 11, -1 },
> +   // The following clashes with the non KHR definition above

We have comments elsewhere like

   // We check for the aliased -OES version in GLES 2

Can you make the comment match that?

> +//   { "glGetPointervKHR", 11, -1 },

Indent.