<p dir="ltr">Seems reasonable. I'm generally a fan of explicitly sized types anyway. R-B</p>
<div class="gmail_extra"><br><div class="gmail_quote">On Sep 30, 2016 4:20 PM, "Timothy Arceri" <<a href="mailto:timothy.arceri@collabora.com">timothy.arceri@collabora.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">These structs will be written to disk as part of the shader cache<br>
so use uint32_t just to be safe.<br>
---<br>
 src/mesa/main/mtypes.h | 20 ++++++++++----------<br>
 1 file changed, 10 insertions(+), 10 deletions(-)<br>
<br>
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h<br>
index d1b0560..efcd50e 100644<br>
--- a/src/mesa/main/mtypes.h<br>
+++ b/src/mesa/main/mtypes.h<br>
@@ -1643,40 +1643,40 @@ struct gl_transform_feedback_varying_<wbr>info<br>
  */<br>
 struct gl_transform_feedback_output<br>
 {<br>
-   unsigned OutputRegister;<br>
-   unsigned OutputBuffer;<br>
-   unsigned NumComponents;<br>
-   unsigned StreamId;<br>
+   uint32_t OutputRegister;<br>
+   uint32_t OutputBuffer;<br>
+   uint32_t NumComponents;<br>
+   uint32_t StreamId;<br>
<br>
    /** offset (in DWORDs) of this output within the interleaved structure */<br>
-   unsigned DstOffset;<br>
+   uint32_t DstOffset;<br>
<br>
    /**<br>
     * Offset into the output register of the data to output.  For example,<br>
     * if NumComponents is 2 and ComponentOffset is 1, then the data to<br>
     * offset is in the y and z components of the output register.<br>
     */<br>
-   unsigned ComponentOffset;<br>
+   uint32_t ComponentOffset;<br>
 };<br>
<br>
<br>
 struct gl_transform_feedback_buffer<br>
 {<br>
-   unsigned Binding;<br>
+   uint32_t Binding;<br>
<br>
-   unsigned NumVaryings;<br>
+   uint32_t NumVaryings;<br>
<br>
    /**<br>
     * Total number of components stored in each buffer.  This may be used by<br>
     * hardware back-ends to determine the correct stride when interleaving<br>
     * multiple transform feedback outputs in the same buffer.<br>
     */<br>
-   unsigned Stride;<br>
+   uint32_t Stride;<br>
<br>
    /**<br>
     * Which transform feedback stream this buffer binding is associated with.<br>
     */<br>
-   unsigned Stream;<br>
+   uint32_t Stream;<br>
 };<br>
<br>
<br>
--<br>
2.7.4<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>
</blockquote></div></div>