Follow-up Comment #1, bug #13848 (project freeciv):
Looks good in most part.
- I don't like "switch" and "case" at same indentation level. I would vote
for half indent step (2/2 = 1) for "case"
- Could you add assert() rule that instead of "assert(FALSE)" assert with
more verbose error message should be used when possible:
switch(value) {
case 1:
case 2:
do_something();
break;
case 3:
assert(value != 3);
break;
}
_______________________________________________________
Reply to this item at:
<http://gna.org/bugs/?13848>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
