<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jul 1, 2014 at 11:06 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>
>> +static void<br>
>> +clear_image_to_zero(GLubyte *dstMap, GLint dstRowStride,<br>
>> +                    GLsizei width, GLsizei height,<br>
>> +                    GLsizei clearValueSize)<br>
>> +{<br>
>> +   while (height-- > 0) {<br>
>> +      memset(dstMap, 0, clearValueSize * width);<br>
>> +      dstMap += dstRowStride;<br>
>> +   }<br>
>> +}<br>
>><br>
><br>
> Technically, this should always work, but you might want to throw in a<br>
> comment about floating-point textures.<br>
<br>
</div>I'm not sure what you mean here. Are you saying that hypothetically<br>
floating-point textures might not represent 0 as zero bytes?<br></blockquote><div><br></div><div>sorry, I got nervous about memsetting floats.  It isn't actually a problem though since floating-point zero is just all zeroes.  It doesn't really matter much.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class=""><br>
> Other than that, looks good to me. (I'll second Ilia's MS comment<br>
> though.)<br>
<br>
</div>Does that count as ‘reviewed-by’? (Yes, in the github branch I've made<br>
the two changes that Ilia suggested.)<br></blockquote><div>Sure.<br><br>Reviewed-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Thanks for the review.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Neil<br>
</font></span></blockquote></div><br></div></div>