On 03/01/2016 05:30 PM, Dave Airlie wrote:
On 4 February 2016 at 01:44, Eduardo Lima Mitev <el...@igalia.com> wrote:
Use QueryInternalFormat instead of QuerySamplesForFormat to obtain the
highest supported sample. QuerySamplesForFormat is to be removed.
---
  src/mesa/main/multisample.c | 10 ++++++----
  1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/mesa/main/multisample.c b/src/mesa/main/multisample.c
index e7783ea..56aa5c3 100644
--- a/src/mesa/main/multisample.c
+++ b/src/mesa/main/multisample.c
@@ -174,10 +174,12 @@ _mesa_check_sample_count(struct gl_context *ctx, GLenum 
target,
      * for <internalformat> then the error INVALID_OPERATION is generated."
      */
     if (ctx->Extensions.ARB_internalformat_query) {
-      GLint buffer[16];
-      int count = ctx->Driver.QuerySamplesForFormat(ctx, target,
-                                                    internalFormat, buffer);
-      int limit = count ? buffer[0] : -1;
+      GLint buffer[16] = {-1};
+      ctx->Driver.QueryInternalFormat(ctx, target, internalFormat,
+                                      GL_SAMPLES, buffer);
+      /* since the query returns samples sorted in descending order,
+       * the first element is the greatest supported sample value */
+      int limit = buffer[0];


Do we allow declarations after code in base mesa code now?

At least one person on mesa-users is reporting a problem with that. I'm diagnosing it with him (which MSVC version, etc).

But don't let that hold you up, Eduardo.

-Brian

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to