[PATCH 2/2] notmuch-reply.c: uncrustify

Tomi Ollila tomi.ollila at iki.fi
Wed Jan 11 07:22:26 PST 2012


On Tue, 10 Jan 2012 08:07:08 -0400, David Bremner <david at tethera.net> wrote:
> From: David Bremner <bremner at debian.org>
> 
> This patch shows the raw result of running uncrustify on notmuch-reply.c.
> The re-indenting of "format_reply" would probably not be desirable.

Nice! It's easy to 'cherry-revert' that part. I wonder is using comments
could be used to preserver formatting in format_reply (and is that is
something wished to be done) i.e. something like:

static const notmuch_show_format_t format_reply = {
    "",
  /* */ "", NULL,
  /* ... */ "", NULL, reply_headers_message_part, ">\n",
  /* ... */ "",
  /* ....... */ NULL,
  /* ....... */ NULL,
  /* ....... */ NULL,
  /* ....... */ reply_part_content,
  /* ....... */ NULL,
  /* ....... */ "",
  /* ... */ "",
  /* */ "", "",


> ---
>  notmuch-reply.c |  160 +++++++++++++++++++++++++-----------------------------
>  1 files changed, 74 insertions(+), 86 deletions(-)
> 
> diff --git a/notmuch-reply.c b/notmuch-reply.c
> index 000f6da..00c9923 100644
> --- a/notmuch-reply.c
> +++ b/notmuch-reply.c
> @@ -32,17 +32,17 @@ reply_part_content (GMimeObject *part);
>  
>  static const notmuch_show_format_t format_reply = {
>      "",
> -	"", NULL,
> -	    "", NULL, reply_headers_message_part, ">\n",
> -	    "",
> -	        NULL,
> -	        NULL,
> -	        NULL,
> -	        reply_part_content,
> -	        NULL,
> -	        "",
> -	    "",
> -	"", "",
> +    "", NULL,
> +    "", NULL, reply_headers_message_part, ">\n",
> +    "",
> +    NULL,
> +    NULL,
> +    NULL,
> +    reply_part_content,
> +    NULL,
> +    "",
> +    "",
> +    "", "",


More information about the notmuch mailing list