I didn't notice that it was declaring a new variable for the for loop
scope instead of setting it to 0. I've fixed all the warnings, even though
the other places looked safe.


On Tue, Feb 12, 2013 at 2:58 PM, Torbjörn Söderstedt <[email protected]>wrote:

> On Tue, Feb 12, 2013 at 11:48 PM, Torbjörn Söderstedt 
> <[email protected]>wrote:
>
>>  The counter isn't reset after counting the number of groups (because the
>> code declared a second variable called "i" that shadowed the first one).
>>
>
> You can add -Wshadow to CPPFLAGS to get warnings about this, but that
> implies fixing all places that currently use shadowing.
>
> _______________________________________________
> Leocad mailing list
> [email protected]
> https://list.gerf.org/listinfo/leocad
>
>
_______________________________________________
Leocad mailing list
[email protected]
https://list.gerf.org/listinfo/leocad

Reply via email to