[Freeciv-Dev] A minor style issue.
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Our coding style guideline gives conflicting signal wrt use of
spaces between "if" and the condition:
if(ptr == NULL)
vs.
if (ptr == NULL)
There are a couple of relevant bits of CodingStyle:
==========
- spaces are inserted before and after operators ("int i, j, k;"
instead of "int a,b,c;" and "if (foo <= bar) c = a + b;" instead
of "if(foo<=bar) c=a+b;")
==========
and
==========
- Comments in conditionals. If you need a comment to show program flow,
it should be below the if or else:
if(is_barbarian(pplayer)) {
x++;
==========
Should we clearly inforce one or the other?
G.
|
|