<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 26, 2014 at 12:50 PM, Jordan Justen <span dir="ltr"><<a href="mailto:jljusten@gmail.com" target="_blank">jljusten@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Sep 23, 2014 at 5:38 PM, Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> wrote:<br>
> Variables are used in later patches to implement<br>
> EXT_framebuffer_multisample_blit_scaled extension.<br>
><br>
> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>><br>
> ---<br>
>  src/mesa/main/mtypes.h | 9 +++++++++<br>
>  1 file changed, 9 insertions(+)<br>
><br>
> diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h<br>
> index 0d50be8..1cb3461 100644<br>
> --- a/src/mesa/main/mtypes.h<br>
> +++ b/src/mesa/main/mtypes.h<br>
> @@ -3608,6 +3608,15 @@ struct gl_constants<br>
>     GLint MaxDepthTextureSamples;<br>
>     GLint MaxIntegerSamples;<br>
><br>
> +   /**<br>
> +    * Layout of sample numbers in a rectangular grid roughly corresponding<br>
> +    * to real sample locations within a pixel. Used by<br>
> +    * GL_EXT_texture_multisample_blit_scaled implementation.<br>
> +    */<br>
> +   GLchar* sample_map_2x;<br>
> +   GLchar* sample_map_4x;<br>
> +   GLchar* sample_map_8x;<br>
<br>
</span>I think this would be better:<br>
   uint8_t SampleMap2x[2]; </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Using a string here seems confusing. The meta code can use asprintf to<br>
build the string.<br></blockquote><div>Yes, I had this thought earlier but wasn't sure. Will fix it now.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The CamelCase name seems to follow the convention of this structure.<br>
<br>
uint8_t doesn't follow the convection of the structure. :) (But, Ian<br>
seems to often try to move us away from GL types when not API facing.)<br>
<br>
Do you think the comment could be improved to help drivers understand<br>
the purpose of the constants? The comment in the 0.2 patch was pretty<br>
clear, but it is i965 specific.<br>
<br>
If you agree to my suggestions, then you should probably send out all<br>
4 patches as a series.<br></blockquote><div>I agree. I'll soon send out the series. Thanks. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-Jordan<br>
<span class=""><br>
>     /** GL_ARB_shader_atomic_counters */<br>
>     GLuint MaxAtomicBufferBindings;<br>
>     GLuint MaxAtomicBufferSize;<br>
> --<br>
> 1.9.3<br>
><br>
</span>> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div></div>