Updated coding style

For code related discussions and questions
Post Reply
Per
Warzone 2100 Team Member
Warzone 2100 Team Member
Posts: 3750
Joined: 03 Aug 2006, 19:39

Updated coding style

Post by 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/bl ... ngStyle.md (new version)

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

Feedback welcome.

Vincent
Trained
Trained
Posts: 103
Joined: 06 Aug 2016, 17:24

Re: Updated coding style

Post by 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.

cybersphinx
Inactive
Inactive
Posts: 1687
Joined: 01 Sep 2006, 19:17

Re: Updated coding style

Post by 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.

Per
Warzone 2100 Team Member
Warzone 2100 Team Member
Posts: 3750
Joined: 03 Aug 2006, 19:39

Re: Updated coding style

Post by 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: 3750
Joined: 03 Aug 2006, 19:39

Re: Updated coding style

Post by 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...

Post Reply