notmuch-reply date format

[Austin Clements]

Quoth Sime Ramov on Jul 15 at  9:35 am:
> Hello,
> 
> * Austin Clements <amdragon at MIT.EDU> [Mon, 14 Jul 2014 09:04:31 -0400]:
> > Assuming the CLI is the right place for you to change this, you
> > probably want to call notmuch_message_get_date, then localtime, then
> > strftime.
> 
> Tried to get this working to no avail. Not a programmer obviously :) I
> cobbled this together from various online sources:
> 
> diff --git a/notmuch-reply.c b/notmuch-reply.c
> index 7c1c809..16cf19c 100644
> --- a/notmuch-reply.c
> +++ b/notmuch-reply.c
> @@ -44,9 +44,15 @@ format_part_reply (mime_node_t *node)
>      int i;
>  
>      if (node->envelope_file) {
> -	printf ("On %s, %s wrote:\n",
> -		notmuch_message_get_header (node->envelope_file, "date"),
> -		notmuch_message_get_header (node->envelope_file, "from"));
> +	struct tm *info;
> +	char dbuf[30];
> +
> +	info = localtime(notmuch_message_get_date(node->envelope_file));

Close.  For whatever reason, localtime takes a pointer to a time_t,
not a time_t (even though time_t is just a number).  You'll need
something like

time_t date;

date = notmuch_message_get_date(node->envelope_file);
info = localtime(&date);

> +
> +	strftime(dbuf, sizeof(dbuf), "%F %R %z", info);
> +	printf ("* %s [%s]:\n",
> +		notmuch_message_get_header(node->envelope_file, "from"), dbuf);
> +
>      } else if (GMIME_IS_MESSAGE (node->part)) {
>  	GMimeMessage *message = GMIME_MESSAGE (node->part);
>  	InternetAddressList *recipients;
> 
> When compiling I get this warning:
> 
> notmuch-reply.c: In function 'format_part_reply':
> notmuch-reply.c:50: warning: passing argument 1 of 'localtime' makes pointer from integer without a cast

Ah, C.  "I'm sure they meant for that number that looks nothing like a
pointer to actually be a pointer that could corrupt memory, disclose
private information, or crash the program.  I'll just convert it for
them real quick..."

> And a core dump when notmuch reply is invoked on a message. Am I even
> close? :) Thanks