Re: [notmuch] [PATCH] notmuch-reply: Use a shorter 'On, X Y wrote:' line

Subject: Re: [notmuch] [PATCH] notmuch-reply: Use a shorter 'On, X Y wrote:' line

Date: Wed, 03 Mar 2010 07:20:47 +0100

To: Sebastian Spaeth, notmuch@notmuchmail.org

Cc:

From: Michal Sojka

On Tue,  2 Mar 2010 13:37:42 +0100, Sebastian Spaeth <Sebastian@SSpaeth.de> wrote:
> Previously, we would output:
> 'On Thu, 25 Feb 2010 14:32:54 +0100, Sebastian Spaeth <Sebastian@SSpaeth.de> wrote:' now it is:
> 'On 2010-02-25, Sebastian Spaeth wrote:'
> 
> In case we don't find a '<' (as indicator for 'Realname <email>'), we still use the whole from address.
> 
> Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
> ---
> This probably shows my lack of C skills quite nicely but it does the job for me.
> 
>  notmuch-reply.c |   21 ++++++++++++++++++---
>  1 files changed, 18 insertions(+), 3 deletions(-)
> 
> diff --git a/notmuch-reply.c b/notmuch-reply.c
> index 98f6442..929572f 100644
> --- a/notmuch-reply.c
> +++ b/notmuch-reply.c
> @@ -288,9 +288,12 @@ notmuch_reply_format_default(void *ctx, notmuch_config_t *config, notmuch_query_
>      GMimeMessage *reply;
>      notmuch_messages_t *messages;
>      notmuch_message_t *message;
> -    const char *subject, *from_addr = NULL;
> +    const char *subject, *from_addr = NULL, *short_from;
>      const char *in_reply_to, *orig_references, *references;
>      char *reply_headers;
> +    time_t date;
> +    struct tm *datetm;
> +    char *datestr;
>  
>      for (messages = notmuch_query_search_messages (query);
>  	 notmuch_messages_has_more (messages);
> @@ -346,10 +349,21 @@ notmuch_reply_format_default(void *ctx, notmuch_config_t *config, notmuch_query_
>  	g_object_unref (G_OBJECT (reply));
>  	reply = NULL;
>  
> +	date = notmuch_message_get_date(message);
> +	datetm = gmtime( &date );
> +	datestr = talloc_array(ctx, char, 11);
> +	strftime(datestr, 11, "%Y-%m-%d", datetm);
> +
> +	/* If from contains '<' (not as first char),
> +	 * only use the preceding real name */
> +	short_from = talloc_strdup(ctx, from_addr);

I do not see, where do you assign a value to from_addr (besides the
initial NULL).

Michal

Thread: