<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Jun 16, 2017 6:08 AM, "Michel Dänzer" <<a href="mailto:michel@daenzer.net">michel@daenzer.net</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 15/06/17 08:56 PM, Marek Olšák wrote:<br>
> From: Marek Olšák <<a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a>><br>
<br>
[...]<br>
<div class="quoted-text"><br>
>     /* For non-black borders... */<br>
> -   if (msamp->BorderColor.ui[0] ||<br>
> -       msamp->BorderColor.ui[1] ||<br>
> -       msamp->BorderColor.ui[2] ||<br>
> -       msamp->BorderColor.ui[3]) {<br>
> +   if ((msamp->BorderColor.ui[0] ||<br>
> +        msamp->BorderColor.ui[1] ||<br>
> +        msamp->BorderColor.ui[2] ||<br>
> +        msamp->BorderColor.ui[3]) &&<br>
> +       /* This is true if wrap modes are using the border color. */<br>
> +       (sampler->wrap_s | sampler->wrap_t | sampler->wrap_r) & 0x1) {<br>
<br>
</div>I'd make this<br>
<br>
   if (/* This is true if wrap modes are using the border color. */<br>
       (sampler->wrap_s | sampler->wrap_t | sampler->wrap_r) & 0x1 &&<br>
       (msamp->BorderColor.ui[0] ||<br>
        msamp->BorderColor.ui[1] ||<br>
        msamp->BorderColor.ui[2] ||<br>
        msamp->BorderColor.ui[3])) {<br>
<br>
No need to test the BorderColor.ui array when the border colour isn't used.<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Done locally.</div><div dir="auto"><br></div><div dir="auto">Marek</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<font color="#888888"><br>
<br>
--<br>
Earthling Michel Dänzer               |               <a href="http://www.amd.com" rel="noreferrer" target="_blank">http://www.amd.com</a><br>
Libre software enthusiast             |             Mesa and X developer<br>
</font></blockquote></div><br></div></div></div>