Re: coding style

Byrial Jensen <[email protected]>
Newsgroups gmane.comp.gnu.xboard.devel
Message-ID <[email protected]>
Den 23-12-2011 03:58, Arun Persaud skrev:
> Hi
>
> how about running "indent -gnu" across all source files and then add a
> pre-commit git hook to make sure that it stays that way?

Well, I would in my opinion make the sources easier to read, but I don't 
have strong feelings about this.

Another thing I could suggest if everything is reformated anyway is to 
get rid of the traditional C style for functions. So instead of

int MyFunc(a, b, c)
	char a, b
	int c
{
   ...

use

int MyFunc(char a, char b, int c)
{
   ...

I don't know if any compilers anymore is dependent of the traditional 
style, I certainly don't know any that is.

But if they exist, they already for some time haven't been able to 
compile xboard anyway because there already is a few uses around of the 
new style (e.g. ics_printf() in backend.c).

Another "new" thing I had like to use, is the const qualifier. It can 
help finding bugs, and it allows for more aggresive optimisation.

Regards
- Byrial
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.