<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 13, 2016 at 1:50 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jason,<br>
<br>
Why do you remove the C99 initializer ? Anything wrong with using it ?<br></blockquote><div><br></div><div>Mostly readability and matching what we did earlier with the other surface state struct.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
On 11 June 2016 at 17:02, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> ---<br>
>  src/intel/isl/isl_surface_state.c | 46 +++++++++++++++++++--------------------<br>
>  1 file changed, 23 insertions(+), 23 deletions(-)<br>
><br>
> diff --git a/src/intel/isl/isl_surface_state.c b/src/intel/isl/isl_surface_state.c<br>
> index ca13175..2026f80 100644<br>
> --- a/src/intel/isl/isl_surface_state.c<br>
> +++ b/src/intel/isl/isl_surface_state.c<br>
> @@ -427,40 +427,40 @@ isl_genX(buffer_fill_state_s)(void *state,<br>
>        assert(num_elements <= (1ull << 27));<br>
>     }<br>
><br>
> -   struct GENX(RENDER_SURFACE_STATE) surface_state = {<br>
> -      .SurfaceType = SURFTYPE_BUFFER,<br>
> -      .SurfaceArray = false,<br>
> -      .SurfaceFormat = info->format,<br>
> -      .SurfaceVerticalAlignment = isl_to_gen_valign[4],<br>
> -      .SurfaceHorizontalAlignment = isl_to_gen_halign[4],<br>
> -      .Height = ((num_elements - 1) >> 7) & 0x3fff,<br>
> -      .Width = (num_elements - 1) & 0x7f,<br>
> -      .Depth = ((num_elements - 1) >> 21) & 0x3f,<br>
> -      .SurfacePitch = info->stride - 1,<br>
> -      .NumberofMultisamples = MULTISAMPLECOUNT_1,<br>
> +   struct GENX(RENDER_SURFACE_STATE) s = { 0 };<br>
> +<br>
> +   s.SurfaceType = SURFTYPE_BUFFER,<br>
</span>Are you sure that this patch builds ? Shouldn't we be using semicolon<br>
and the end of the above line (plus all the ones below) ?<br></blockquote><div><br></div><div>Rebase fail!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
-Emil<br>
</font></span></blockquote></div><br></div></div>