<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 25, 2015 at 8:38 PM, Nanley Chery <span dir="ltr"><<a href="mailto:nanleychery@gmail.com" target="_blank">nanleychery@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Tue, Aug 25, 2015 at 5:38 PM, Dave Airlie <span dir="ltr"><<a href="mailto:airlied@gmail.com" target="_blank">airlied@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">From: Dave Airlie <<a href="mailto:airlied@redhat.com" target="_blank">airlied@redhat.com</a>><br>
<br>
commit 26c549e69d12e44e2e36c09764ce2cceab262a1b<br>
Author: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com" target="_blank">nanley.g.chery@intel.com</a>><br>
Date:   Fri Jul 31 10:26:36 2015 -0700<br>
<br>
    mesa/formats: remove compressed formats from matching function<br>
<br>
caused a regression in my CTS testing, this looks like a clear<br>
thinko.<br>
<br>
sSigned-off-by: Dave Airlie <<a href="mailto:airlied@redhat.com" target="_blank">airlied@redhat.com</a>><br></blockquote></span><div>You have an extra 's' at the beginning of this line. </div><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
---<br>
 src/mesa/main/formats.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/main/formats.c b/src/mesa/main/formats.c<br>
index 0d88c0b..f217adc 100644<br>
--- a/src/mesa/main/formats.c<br>
+++ b/src/mesa/main/formats.c<br>
@@ -2046,7 +2046,7 @@ _mesa_format_matches_format_and_type(mesa_format mesa_format,<br>
    case MESA_FORMAT_X8R8G8B8_SRGB:<br>
       return GL_FALSE;<br>
    default:<br>
-      assert(_mesa_is_format_compressed(format));<br>
+      assert(_mesa_is_format_compressed(mesa_format));<br>
       if (error)<br>
          *error = GL_INVALID_ENUM;<br>
    }<br>
<span><font color="#888888">--<br>
2.4.3<br><br></font></span></blockquote><div><br></div></span><div>Thanks for finding this error! FWIW,</div><div><br></div><div>Reviewed-by: Nanley Chery <nanley.g.chery@.<a href="http://intel.com" target="_blank">intel.com</a>> </div></div><br></div></div>
</blockquote></div>Please ignore the extra '.' after the '@' ;-).</div></div>