On Thu, Mar 08, 2007 at 11:46:53PM -0500, David Hampton wrote:
> There's been some discussion on IRC about using the indent program to
> reformat the sources into something consistent from file to file.  I'd
> like to propose the following options as a starting point for
> discussion.  I think some of these options will be agreed upon by all,
> and I'm sure others will be contentious.  :-)
> 
> Indentation
> 
> -nut  No tabs. Indentation is done with spaces.
> -i4   Add four spaces for each indent level
> -ci4  Continuation lines are indented by four spaces
> -cli4 Case labels are indented by four spaces
> -ppi4 Nested pre-processor defines are indented by four spaces
> -nbc  Do not force newlines after commas in declarations (default)
> -nbfda        Don´t put each argument in a function declaration
>         on a separate line (default)
> -lp   Line up continued lines at parentheses (default)
> -psl  Put the type of a procedure on the line before its name.
> -bbo  Prefer to break long lines before boolean operators.
> 
> Blank Lines
> 
> -nsob Do not swallow optional blank lines (default)
> -bad  Force a blank line after a declaration.
> -bbb  Force a blank line before a block comment.
> -bap  Force blank lines after procedure bodies.
> 
> Comments
> 
> -fca  Reformat all comments except those starting in column 1
> -fc1  Reformat comments starting in column 1
> -sc   Continuation lines in a comments start with a '*'
> 
> Statements
> 
> -npcs Do not put space after the function in function calls.
> -nprs Do not put a space after every ´(´ and before every ´)´.
> 
> -ncs  Do not put a space after cast operators.
> -saf  Put a space after each for.
> -sai  Put a space after each if.
> -saw  Put a space after each while.
> 
> -brs  Put braces on struct declaration line.
> -br   Put braces on line with if, etc.
> -ce   Cuddle else and preceeding `}´.
> -cdw  Cuddle while of do {} while; and preceeding `}´.
> 
> Other
> 
> -l80  Line width of 80.
> 
> Attached is a small source file formatted with the above options.
> 
> Let the discussion begin. :-)
> 
> Of the above list of options, the ones I feel strongly about are lining
> up continued lines at parentheses (-lp), the space-after-keyword group
> (saf, -sai, -saw), and the cuddling-of-braces group (-br, -ce, -cdw).  I
> feel most strongly about this last group.  The parts of an if/then/else
> statement are clearly delineated by the indentation of the keywords vs.
> the indentation of the code blocks.  I believe it a complete waste to
> additionally add vertical separation my making the "else" take up three
> lines.  One for a closing parenthesis, one for the word "else", and one
> for an opening parenthesis.  Maybe that's just me.
> 
> What do the rest of the developers think?

I like it.  It's basically K&R, but with -bad -psl and some
comment-formatting.

I would recommend adding -ss, and I would have defaulted to -cli0,
which is also K&R, but I don't feel strongly about it.

-chris

> 
> David
>  
> 


> _______________________________________________
> gnucash-devel mailing list
> gnucash-devel@gnucash.org
> https://lists.gnucash.org/mailman/listinfo/gnucash-devel

_______________________________________________
gnucash-devel mailing list
gnucash-devel@gnucash.org
https://lists.gnucash.org/mailman/listinfo/gnucash-devel

Reply via email to