Changes between Version 11 and Version 12 of CodeStyle

Timestamp:

12/08/16 22:35:03 (7 years ago)

Author:

Steffan Karger

Comment:

--

CodeStyle

@@ -86 +86 @@
 This makes the intent immediately clear to the reader / reviewer.
 
-= GNU Indent rules =
-to get the above effect, use
-{{{
-gindent -kr -bl -nce -nbc -l80 file.c
-}}}
-(though other options might be needed, work in progress)
-
-Indent likes to produce artifacts, though, because it really really likes putting the return type on the same line as the function name, which creates long and messy declarations for many functions in tun.c (my test candidate).
-
-A slightly less brutal tool (because it can be told to just leave certain constructs alone, while changing others) seems to be
+= Crustify rules =
+The most fit tool for our purpose (because it can be told to just leave certain constructs alone, while changing others) seems to be
 
 {{{
@@ -101 +93 @@
 }}}
 
-... but that one has a zillion of options (600)... some options for a start
+... but it has a zillion of options (600)... our config-in-the-making:
 {{{
+# Use Allman-style, space-only
 indent_columns=4
-indent_with_tabs=2
+indent_with_tabs=0
+align_with_tabs=false
 indent_braces=false
 indent_else_if=false
-indent_switch_case=2
-indent_label=-2
-sp_before_sparen=add
-sp_inside_sparen=remove
 nl_if_brace=add
 nl_brace_else=add
@@ -116 +106 @@
 nl_else_brace=add
 nl_else_if=remove
+sp_func_proto_paren=Remove
+sp_func_def_paren=Remove
+sp_func_call_paren=Remove
+
+# Do not put spaces between the # and preprocessor statements
+pp_space=remove
+
+cmt_convert_tab_to_spaces=true
+indent_switch_case=4
+indent_label=1
+sp_before_sparen=add
+sp_inside_sparen=remove
+sp_cond_colon=add
+sp_cond_question=add
+sp_bool=add
+sp_else_brace=add
+sp_brace_else=add
+pos_arith=Lead
 nl_func_type_name=add
+nl_before_case=true
+nl_assign_leave_one_liners=true
+nl_enum_leave_one_liners=true
+nl_brace_fparen=add
+nl_max=4
+nl_after_func_proto=2
 code_width=80
+ls_code_width=false
 mod_full_brace_if=add
 mod_full_brace_if_chain=false
 mod_add_long_ifdef_endif_comment=20
-mod_add_long_ifdef_else_comment
+mod_add_long_ifdef_else_comment=5
+mod_remove_extra_semicolon=true
 cmt_c_nl_end=true
 cmt_star_cont=true
-pp_indent=add 
+
+# Use "char **a"-style pointer stars
+sp_before_ptr_star=Add
+sp_between_ptr_star=Remove
+sp_after_ptr_star=Remove
+
 }}}