<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 27, 2014 at 5:41 PM, Chris Forbes <span dir="ltr"><<a href="mailto:chrisf@ijw.co.nz" target="_blank">chrisf@ijw.co.nz</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Sat, Jun 28, 2014 at 10:19 AM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>

<br>
> @@ -653,135 +613,6 @@ _swrast_texture_span( struct gl_context *ctx, SWspan *span )<br>
</div>[snip]<br>
<div class=""><br>
> -<br>
> -   /*<br>
> -    * Must do all texture sampling before combining in order to<br>
> -    * accomodate GL_ARB_texture_env_crossbar.<br>
> -    */<br>
> -   for (unit = 0; unit < ctx->Const.MaxTextureUnits; unit++) {<br>
> -      const struct gl_texture_unit *texUnit = &ctx->Texture.Unit[unit];<br>
> -      if (texUnit->_Current &&<br>
> -          texUnit->_CurrentCombine->ModeRGB != GL_BUMP_ENVMAP_ATI) {<br>
<br>
</div>You're ripping out the non-bump case here too...<br></blockquote><div><br></div><div>Good Catch, thanks.<br></div><div>--Jason Ekstrand<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5"><br>
> -         const GLfloat (*texcoords)[4] = (const GLfloat (*)[4])<br>
> -            span->array->attribs[VARYING_SLOT_TEX0 + unit];<br>
> -         const struct gl_texture_object *curObj = texUnit->_Current;<br>
</div></div></blockquote></div><br></div></div>