On Fri, Nov 6, 2015 at 8:27 AM, Juan A. Suarez Romero
<jasua...@igalia.com> wrote:
> Replace the current loop by a direct call to _mesa_fls() function.
>
> It also fixes an implicit bug in the current code where num_textures
> seems to be one value less than it should be when sh->Program->SamplersUsed > 
> 0.
>
> For instance, num_textures is 0 instead of 1 when
> sh->Program->SamplersUsed is 1.

Looks good to me, and we use _mesa_fls elsewhere to do this same calculation.

Reviewed-by: Matt Turner <matts...@gmail.com>

Jason, was there some reason we weren't doing this? I'm confused why
we would have had a one-line comment and a 4-line loop when the all we
needed to do was call one functon?
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to