Bug Tracker – Bug 875

Cleanup source with Artistic Style

Last modified: 2016-09-16 00:27:55 CDT
Bug 875 - (ARRAY(0x4d5fcf8)) Cleanup source with Artistic Style
(ARRAY(0x4d5fcf8))
Cleanup source with Artistic Style
Status: NEW
Product: Odamex
Classification: Unclassified
Component: Server & Client
(old) 0.6-dev
All All
: P1 enhancement
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2012-07-15 17:09:49 CDT by Alexander Mayfield
Modified: 2016-09-16 00:27 CDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Mayfield 2012-07-15 17:09:49 CDT
Quoth the website: "Artistic Style is a source code indenter, formatter, and beautifier for the C, C++, C# and Java programming languages."  Right now, our source tree contains many different source formatting styles.  This program can be used to clean things up.

Here is a config file that I came up with that mirrors our own code formatting guidelines:

style=allman
indent=tab=4
indent-col1-comments
pad-header
unpad-paren
convert-tabs
align-pointer=type
align-reference=name
lineend=linux
suffix=none

We can keep these settings in a file called astylerc in the root of our source tree.

However, this cleanup is not something we should do lightly.  If anyone has any outstanding patches, they will be much more difficult to apply after the cleanup.  There is also the small possibility that this cleanup might break functioning code for some obscure reason, so we ought to heavily test the result.
Comment 1 Russell Rice 2012-07-16 04:01:26 CDT
Just as a note, there is an AStyle plugin that comes with codeblocks too