<div dir="ltr">On Sun, Mar 26, 2017 at 5:51 PM, Edward O'Callaghan <span dir="ltr"><<a href="mailto:funfunctor@folklore1984.net" target="_blank">funfunctor@folklore1984.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">V.1:<br>
We memset number of elements without multiplication by the<br>
element size.<br>
<br>
V.2:<br>
We explicitly set each member to -1 over using a confusing<br>
memset().<br>
<br>
Signed-off-by: Edward O'Callaghan <<a href="mailto:funfunctor@folklore1984.net">funfunctor@folklore1984.net</a>><br>
---<br>
 src/mesa/main/formatquery.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/main/formatquery.c b/src/mesa/main/formatquery.c<br>
index 598d34d..6aa57ec 100644<br>
--- a/src/mesa/main/formatquery.c<br>
+++ b/src/mesa/main/formatquery.c<br>
@@ -1564,7 +1564,7 @@ _mesa_GetInternalformati64v(<wbr>GLenum target, GLenum internalformat,<br>
     * no pname can return a negative value, we fill params32 with negative<br>
     * values as reference values, that can be used to know what copy-back to<br>
     * params */<br>
-   memset(params32, -1, 16);<br>
+   for (i = 0; i < realSize; i++) params32[i] = -1;<br></blockquote><div><br></div><div>We'd generally write that for loop on two lines.<br><br>for (i = 0; i < realSize; i++)<br>   params32[i] = -1;<span class=""><br><br></span></div><div>-Brian<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
    /* For GL_MAX_COMBINED_DIMENSIONS we need to get back 2 32-bit integers,<br>
     * and at the same time we only need 2. So for that pname, we call the<br>
<span class="gmail-HOEnZb"><font color="#888888">--<br>
2.9.3<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>