Talk:Style Guidelines

Revision as of 15:34, 4 August 2009 by Isilkor (talk | contribs)

Responses so far: Günther:

  • local headers before system headers
  • don't forbid public data members
  • <!Guenther> Demanding Doygen comments is also a very radical change
<!Guenther> Doxygen is only really usefull if you are generating API docs that are intended to be read without looking at the source code
<!Guenther> That's not a use case for us: our API is C4Script, not some C++ API

PeterW:

  • order switch members by numeric value

Afterthoughts Isilkor:

  • Use forward declarations instead of including the header inside other headers, if possible.

Opinions:

  • Use anonymous namespaces instead of static: In favor: ck?, Isilkor; Against: Günther, PeterW
  • Doxygen comments: In favor:; Against: Günther, PeterW