<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jul 1, 2014 at 10:09 AM, Neil Roberts <span dir="ltr"><<a href="mailto:neil@linux.intel.com" target="_blank">neil@linux.intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> writes:<br>
<br>
>> +   texImages[0] = _mesa_select_tex_image(ctx, texObj, texObj->Target,<br>
>> level);<br>
>><br>
><br>
> Do you want this inside an else block?<br>
<br>
</div>I think it's quite a common idiom in Mesa to handle the special cases as<br>
a series of if-statements with a return upfront before handling the<br>
normal case at the end without an else. But yes, in this instance it<br>
does look a bit odd because the there is only one special case and the<br>
normal case is very short, so I don't mind changing it.<br></blockquote><div><br></div><div>Sorry, I missed the return.  In that case, I don't think it matters; feel free to leave it alone.<br></div><div>--Jason<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Regards,<br>
- Neil<br>
</blockquote></div><br></div></div>