<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 2, 2017 at 1:09 PM, Chad Versace <span dir="ltr"><<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</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 Fri 26 May 2017, Jason Ekstrand wrote:<br>
> ---<br>
>  src/intel/isl/isl.c | 19 +++++++++++++++++++<br>
>  src/intel/isl/isl.h |  4 ++++<br>
>  2 files changed, 23 insertions(+)<br>
><br>
> diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c<br>
> index 321850e..4eec2fd 100644<br>
> --- a/src/intel/isl/isl.c<br>
> +++ b/src/intel/isl/isl.c<br>
> @@ -267,6 +267,25 @@ isl_tiling_get_info(const struct isl_device *dev,<br>
>     return true;<br>
>  }<br>
><br>
> +bool<br>
> +isl_color_value_is_zero_one(<wbr>union isl_color_value value,<br>
> +                            enum isl_format format)<br>
> +{<br>
> +   if (isl_format_has_int_channel(<wbr>format)) {<br>
> +      for (unsigned i = 0; i < 4; i++) {<br>
<br>
</span>This patch assumes the all four channels of isl_color_value are<br>
initialized, even if the isl_format has fewer than four channels.<br>
This smells like the seed of a hard-to-reproduce bug.<br>
<br>
Since it's not possible today to query the number of channels in an<br>
isl_format, please at least add a scary comment on the function about<br>
this.<br>
</blockquote></div><br></div><div class="gmail_extra">Yeah... Turns out this patch isn't actually needed for anything.  I'll just drop it for now.<br></div></div>