Module: Mesa
Branch: master
Commit: b48c42cd1f81bf9a411d19aea2660d75622a1350
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b48c42cd1f81bf9a411d19aea2660d75622a1350

Author: Alejandro PiƱeiro <apinhe...@igalia.com>
Date:   Thu May  5 11:27:05 2016 +0200

mesa/formatquery: add a comment to clarify INTERNALFORMAT_PREFERRED

The comment clarifies that the driver is called only to try to get
a preferred internalformat, and that it was already checked if the
format is supported or not.

Acked-by: Eduardo Lima <el...@igalia.com>
Acked-by: Antia Puentes <apuen...@igalia.com>

Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>

---

 src/mesa/main/formatquery.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/mesa/main/formatquery.c b/src/mesa/main/formatquery.c
index 215c14f..1f21d17 100644
--- a/src/mesa/main/formatquery.c
+++ b/src/mesa/main/formatquery.c
@@ -902,7 +902,10 @@ _mesa_GetInternalformativ(GLenum target, GLenum 
internalformat, GLenum pname,
        *     format for representing resources of the specified 
<internalformat> is
        *     returned in <params>.
        *
-       * Therefore, we let the driver answer.
+       * Therefore, we let the driver answer. Note that if we reach this
+       * point, it means that the internalformat is supported, so the driver
+       * is called just to try to get a preferred format. If not supported,
+       * GL_NONE was already returned and the driver is not called.
        */
       ctx->Driver.QueryInternalFormat(ctx, target, internalformat, pname,
                                       buffer);

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

Reply via email to