unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] uncrustify.cfg: label indent, some known types, not, # and ##
@ 2012-01-24 20:55 Tomi Ollila
  2012-01-25 11:52 ` David Bremner
  0 siblings, 1 reply; 2+ messages in thread
From: Tomi Ollila @ 2012-01-24 20:55 UTC (permalink / raw)
  To: notmuch; +Cc: Tomi Ollila

Adjusted some uncrustify variables to get closer to prevailing style:

* Label indent (for goto) relative to current indentation.
* Eegistered GMimeObject and mime_node_t being as types.
* Space after ! (not) operator.
* No space after 'stringify' (#) preprosessor token.
* No spacing change around ## (option not versatile enough).

There are at least 3 cases where attention needs to be paid:

* If there is newline between function name and open paren in function
  call, the paren (and args) are indented too far right.
* #define HOUR (60 *MINUTE) -- i.e. no space after star (*).
* void (*foo)(args) -- i.e no space between (name) and (args).
---
 devel/uncrustify.cfg |   11 +++++++++--
 1 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/devel/uncrustify.cfg b/devel/uncrustify.cfg
index 1dbc5e4..d8075ba 100644
--- a/devel/uncrustify.cfg
+++ b/devel/uncrustify.cfg
@@ -16,8 +16,7 @@ input_tab_size		= 8		# original tab size
 output_tab_size		= 8		# new tab size
 indent_columns		= 4
 
-indent_label		= 2		# pos: absolute col, neg: relative column
-
+indent_label		= -2		# pos: absolute col, neg: relative column
 
 #
 # inter-symbol newlines
@@ -55,6 +54,11 @@ nl_after_struct = 0
 # mod_full_brace_do	= remove	# "do a--; while ();" vs "do { a--; } while ();"
 # mod_full_brace_while	= remove	# "while (a) a--;" vs "while (a) { a--; }"
 
+#
+# Extra types used in notmuch source.
+# (add more on demand)
+
+type GMimeObject mime_node_t
 
 #
 # inter-character spacing options
@@ -63,6 +67,9 @@ nl_after_struct = 0
 sp_before_ptr_star	= force
 sp_between_ptr_star	= remove
 sp_after_ptr_star	= remove
+sp_not			= force
+sp_pp_concat		= ignore	# XXX 'remove' drops leading space also
+sp_pp_stringify		= remove
 
 # sp _return_paren	= force		# "return (1);" vs "return(1);"
 sp_sizeof_paren		= force		# "sizeof (int)" vs "sizeof(int)"
-- 
1.7.8.2

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] uncrustify.cfg: label indent, some known types, not, # and ##
  2012-01-24 20:55 [PATCH] uncrustify.cfg: label indent, some known types, not, # and ## Tomi Ollila
@ 2012-01-25 11:52 ` David Bremner
  0 siblings, 0 replies; 2+ messages in thread
From: David Bremner @ 2012-01-25 11:52 UTC (permalink / raw)
  To: Tomi Ollila, notmuch; +Cc: Tomi Ollila

On Tue, 24 Jan 2012 22:55:59 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrote:
> Adjusted some uncrustify variables to get closer to prevailing style:
> 

pushed.

d

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-01-25 11:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-24 20:55 [PATCH] uncrustify.cfg: label indent, some known types, not, # and ## Tomi Ollila
2012-01-25 11:52 ` David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).