Per Inge Mathisen schreef: > On 4/5/07, Giel van Schijndel <[EMAIL PROTECTED]> wrote: >> * remove some else branches to inrease code readability (code itself >> should function >> exactly the same) > > Just a nitpick: You should always use curly brackets after an if > statement. If the single statement following the if is a poorly made > macro, bad things can happen. > > Also see http://wz2100.net/wiki/development:coding_style Of course one should never use poorly written macros. And personally I always prefer functions or (inlined if necessary) above macros.
But apart from that for the sake of consistency using the above style would probably do well, so I'll (try to) think of that with next changes I make. -- Giel
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev