[Mesa-dev] [PATCH 01/10] tnl: add some comments in render_line_loop code

Brian Paul brianp at vmware.com
Fri Oct 16 15:20:32 PDT 2015


On 10/16/2015 04:14 PM, Matt Turner wrote:
> On Fri, Oct 16, 2015 at 2:25 PM, Brian Paul <brianp at vmware.com> wrote:
>> And remove '(void) flags' line which is not needed.
>> ---
>>   src/mesa/tnl/t_vb_rendertmp.h | 5 +++--
>>   1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/mesa/tnl/t_vb_rendertmp.h b/src/mesa/tnl/t_vb_rendertmp.h
>> index 44dee76..26a1695 100644
>> --- a/src/mesa/tnl/t_vb_rendertmp.h
>> +++ b/src/mesa/tnl/t_vb_rendertmp.h
>> @@ -124,19 +124,19 @@ static void TAG(render_line_loop)( struct gl_context *ctx,
>>      GLuint i;
>>      LOCAL_VARS;
>>
>> -   (void) flags;
>> -
>>      INIT(GL_LINE_LOOP);
>>
>>      if (start+1 < count) {
>>         if (TEST_PRIM_BEGIN(flags)) {
>>           RESET_STIPPLE;
>> +         /* draw the first line from v[0] to v[1] */
>>            if (ctx->Light.ProvokingVertex == GL_LAST_VERTEX_CONVENTION_EXT)
>>               RENDER_LINE( ELT(start), ELT(start+1) );
>>            else
>>               RENDER_LINE( ELT(start+1), ELT(start) );
>>         }
>>
>> +      /* draw lines from v[1] to V[n-1] */
>
> Presumably lower case v[n-1] here.

I'll fix that before pushing.

-Brian



More information about the mesa-dev mailing list