Le dimanche 6 janvier 2013 17:03:28, grischka a écrit :
> Other candidates for reversion (OTOH):
> -
> http://repo.or.cz/w/tinycc.git/commitdiff/fc574f14984d11f1ead50560d1bdc5ae
> 0eaf6d8d The headers from include are copied to win32/include during
> install

I'm not sure to understand why _STATIC_ASSERT was modified. Are you just 
claiming the fix is subobptimal because of the code duplication or the fix is 
useless?

> 
> -
> http://repo.or.cz/w/tinycc.git/commitdiff/943574aba54713ca4a17fe33aadde5ab
> ee233b53 Obviously (if you look at the code) this was not a bug but
> intended behavior.

Indeed. I followed the chain of changes when I noticed that patch and it led 
to your commit f366cb20fe06396609a5438756c3c49662b7316c. However, I didn't 
read into details the content of the for loops at that time. I just assumed a 
bug was noticed by someone and that was the fix. Now, I see the "if (0 == *p)" 
and realize it was indeed intentional. Reverted.

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Tinycc-devel mailing list
Tinycc-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/tinycc-devel

Reply via email to