<div class="gmail_quote">On Tue, Apr 26, 2011 at 4:57 PM, Brian Paul <span dir="ltr">&lt;<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div>On 04/26/2011 04:57 AM, Marek Olšák wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
this series implements GL_EXT_texture_shared_exponent.<br>
<br>
The file rgb9e5.h has all its functions inline, because it&#39;s shared<br>
between classic and gallium. In that file, there are functions<br>
rgb9e5_to_float3 and float3_to_rgb9e5, which are used pretty much<br>
everywhere.<br>
</blockquote>
<br></div>
I think I&#39;d opt to put the code for this in gallium/util and include it in Mesa with a wrapper header.  In the future I think we&#39;re more likely to use additional Gallium utility code in Mesa than vice versa.</blockquote>


<div><br></div></div>Here&#39;s a follow-up patch incorporating most of the suggestions:<br><br><a href="http://cgit.freedesktop.org/%7Emareko/mesa/commit/?id=1ac0d968df40dcb6911567bec1d8ec4ebcb6a2d2" target="_blank">http://cgit.freedesktop.org/~mareko/mesa/commit/?id=1ac0d968df40dcb6911567bec1d8ec4ebcb6a2d2</a><br>


<br>Not sure how you would like the wrapper header to look like. FWIW I don&#39;t think it&#39;s terribly important though.<br><br>Marek<br>