mesa/formatquery: expand NUM_SAMPLE_COUNTS OpenGL ES comment
authorAlejandro Piñeiro <apinheiro@igalia.com>
Fri, 6 May 2016 14:13:26 +0000 (16:13 +0200)
committerAlejandro Piñeiro <apinheiro@igalia.com>
Fri, 3 Jun 2016 05:38:25 +0000 (07:38 +0200)
For ES 3.0 NUM_SAMPLE_COUNTS spec points that some formats will be
always zero. But on ES 3.1 can be different to zero.

The current code is correctly checking exactly against version 3.0,
but the comment only mentions 3.0 spec. It is clearer mentioning both.

v2: better wording on the comment (Ian Romanick)

Acked-by: Eduardo Lima <elima@igalia.com>
Acked-by: Antia Puentes <apuentes@igalia.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/mesa/main/formatquery.c

index 1f21d17fff6c13a4a5a65bd29cef7d4b2daefeca..f524619d98ea28f32e3434ae51f35b32672a5621 100644 (file)
@@ -877,6 +877,9 @@ _mesa_GetInternalformativ(GLenum target, GLenum internalformat, GLenum pname,
        *     "Since multisampling is not supported for signed and unsigned
        *     integer internal formats, the value of NUM_SAMPLE_COUNTS will be
        *     zero for such formats.
+       *
+       * Since OpenGL ES 3.1 adds support for multisampled integer formats, we
+       * have to check the version for 30 exactly.
        */
       if (pname == GL_NUM_SAMPLE_COUNTS && ctx->API == API_OPENGLES2 &&
           ctx->Version == 30 && _mesa_is_enum_format_integer(internalformat)) {