Updated coding style

For code related discussions and questions

Updated coding style

Postby Per » 26 Aug 2017, 22:10

Our wiki is showing signs of old age and general lack of maintenance, and I am thinking we should take it down at some point, as there is not much useful content there anymore. One of the things that's still there is our old coding style guide, which I've now reformatted and updated (mostly by removing stuff) a bit with github markdown.

See https://github.com/perim/warzone2100/blob/master/doc/CodingStyle.md (new version)

It is currently http://developer.wz2100.net/wiki/CodingStyle (old version)

Feedback welcome.
Per
Warzone 2100 Team Member
Warzone 2100 Team Member
 
Posts: 3649
Joined: 03 Aug 2006, 19:39

Re: Updated coding style

Postby Vincent » 27 Aug 2017, 21:17

Is there a .clang-format file to automate style formatting with clang format ? It would make a lot easier to ensure consistent formatting when submitting PR.
Vincent
Trained
Trained
 
Posts: 103
Joined: 06 Aug 2016, 17:24

Re: Updated coding style

Postby cybersphinx » 27 Aug 2017, 22:54

The linked doxygen docs are over three years out of date (I don't remember how they were supposed to be built, but I guess it doesn't work anymore).
We want information... information... information.
cybersphinx
Inactive
Inactive
 
Posts: 1671
Joined: 01 Sep 2006, 19:17

Re: Updated coding style

Postby Per » 27 Aug 2017, 23:51

Vincent wrote:Is there a .clang-format file to automate style formatting with clang format ? It would make a lot easier to ensure consistent formatting when submitting PR.

I don't know anyone who has that, I am quite happy with astyle, but if you make one, feel free to make a PR with it against the CodingStyle doc.
Per
Warzone 2100 Team Member
Warzone 2100 Team Member
 
Posts: 3649
Joined: 03 Aug 2006, 19:39

Re: Updated coding style

Postby Per » 27 Aug 2017, 23:52

cybersphinx wrote:The linked doxygen docs are over three years out of date (I don't remember how they were supposed to be built, but I guess it doesn't work anymore).

Was it ever used / useful? I guess since nobody has noticed that it is three years out of date, the answer is no...
Per
Warzone 2100 Team Member
Warzone 2100 Team Member
 
Posts: 3649
Joined: 03 Aug 2006, 19:39


Return to Coding