> Is the lack of extra curly braces now a warning, and therefore we
> should code accordingly?

Yeah, in gcc-4.3 it warns that it is ambiguous when 'if' the 'else'
corresponds to.  My personal feeling is that for compound clauses which fit
on one screen height it is pretty clear provided you have decent indenting,
but it is an easy enough change.

I'm not in a hurry to fix the few remaining places where it warns -- I
assume they'll get fixed by attrition when gcc-4.3 becomes the base of newer
linux installations.

Also, much to Roy's chagrin I'm sure, it warns about uber-complex clauses
when you've got compounded && || etc... and are relying on the evaluation
rules for correctness.  I've always leaned a little far to the superfluous
()-side because I'm nervous I'll screw one up.

-Ben


-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Libmesh-devel mailing list
Libmesh-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libmesh-devel

Reply via email to