RE : RE : RE : RE : RE : [lwip-devel] Can we remove pbuf_init()

[Frédéric BERNON]

>Is it a warning that can be turned off in your compiler?

No, it can't. And, except in this case, I think it's most of time legitimate... 
I also note yesterday, using Simon's msvc6 port, that with "Level 4 (/W4)" 
warning level, I got same warnings :

\OPENSOURCE\LwIP\src\api\api_lib.c(728) : warning C4127: conditional expression 
is a constant

Something like ~480 warning in lwIP (with each LWIP_ASSERT, LWIP_ERROR, 
etc...). But with VC++, we can set the warning level to /W3 to avoid that...
  
====================================
Frédéric BERNON 
HYMATOM SA 
Chef de projet informatique 
Microsoft Certified Professional 
Tél. : +33 (0)4-67-87-61-10 
Fax. : +33 (0)4-67-70-85-44 
Email : address@hidden 
Web Site : http://www.hymatom.fr 
====================================
P Avant d'imprimer, penser à l'environnement
 


-----Message d'origine-----
De : address@hidden [mailto:address@hidden De la part de Jonathan Larmour
Envoyé : lundi 30 juillet 2007 18:30
À : lwip-devel
Objet : Re: RE : RE : RE : RE : [lwip-devel] Can we remove pbuf_init()


Frédéric BERNON wrote:
> http://vivekkutal.blogspot.com/2006/03/do-while0.html give a explain 
> of the semicolon problem you talk (not the same I thought).

Ah yes, of course, that makes better sense. It's always the form I use for 
definite.

> Now, I
> understand, even if with current lwIP coding style (where we put 
> brackets for each "if", "while", etc... even with one line in the 
> "block"), I think such problem can't happen... Thank you for this 
> explain...

Although it only takes one instance not to, so defensive coding would imply 
we still should. Is it a warning that can be turned off in your compiler?

Jifl
-- 
eCosCentric Limited      http://www.eCosCentric.com/     The eCos experts
Barnwell House, Barnwell Drive, Cambridge, UK.       Tel: +44 1223 245571
Registered in England and Wales: Reg No 4422071.
------["The best things in life aren't things."]------      Opinions==mine


_______________________________________________
lwip-devel mailing list
address@hidden http://lists.nongnu.org/mailman/listinfo/lwip-devel

Frédéric BERNON.vcf
Description: Frédéric BERNON.vcf