<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Dec 3, 2013 at 4:28 PM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.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="im">On 12/03/2013 02:56 PM, Courtney Goeltzenleuchter wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hi Brian,<br>
<br>
I've made all the recommended changes.<br>
<br>
I also added one, a test that the Driver.TextureView != NULL before<br>
calling. This lets me test the non DD functionality with the<br>
MESA_EXTENSION_OVERRIDE=+GL_<u></u>ARB_texture_view<br>
<br>
I've also rebased to latest master (no changes required). Whole stream<br>
can be found at:<br>
<a href="https://github.com/courtney-lunarg/mesa/tree/texture_view-rc6" target="_blank">https://github.com/courtney-<u></u>lunarg/mesa/tree/texture_view-<u></u>rc6</a><br></div>
<<a href="https://urldefense.proofpoint.com/v1/url?u=https://github.com/courtney-lunarg/mesa/tree/texture_view-rc6&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=lGQMzzTgII0I7jefp2FHq7WtZ%2BTLs8wadB%2BiIj9xpBY%3D%0A&m=StSJ4O%2Fa%2B9Zp%2FeTtJIDzJ1NxrKtDXrENaO%2FoFA9gY1k%3D%0A&s=0439e3fc50d195734c13b4c5e22024f034c27b7e657c8f79e08993b98f117518" target="_blank">https://urldefense.<u></u>proofpoint.com/v1/url?u=https:<u></u>//github.com/courtney-lunarg/<u></u>mesa/tree/texture_view-rc6&k=<u></u>oIvRg1%2BdGAgOoM1BIlLLqw%3D%<u></u>3D%0A&r=<u></u>lGQMzzTgII0I7jefp2FHq7WtZ%<u></u>2BTLs8wadB%2BiIj9xpBY%3D%0A&m=<u></u>StSJ4O%2Fa%2B9Zp%<u></u>2FeTtJIDzJ1NxrKtDXrENaO%<u></u>2FoFA9gY1k%3D%0A&s=<u></u>0439e3fc50d195734c13b4c5e22024<u></u>f034c27b7e657c8f79e08993b98f11<u></u>7518</a>><div class="im">
<br>
<br>
Let me know if you need me to post the patches to the mail list or not.<br>
</div></blockquote>
<br>
The series LGTM.  Thanks.  Reviewed-by: Brian Paul <<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>><br>
<br>
Do you need someone to push to master for you?</blockquote><div><br></div><div>Yes please!</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>

<br>
-Brian<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Courtney Goeltzenleuchter<br><div>LunarG</div><div><img src="http://media.lunarg.com/wp-content/themes/LunarG/images/logo.png" width="96" height="65"><br>
</div></div>
</div></div>