Re: [9front] [PATCH] nedmail: only display most useful headers

[unobe@cpan.org]

I like it, thanks phil!  One other that I would add (like CC, only if
set) is inreplyto.  But maybe that's not most useful to you and
others.

Quoth phil9 <telephil9@gmail.com>:
> Hi,
> 
> Below patch limits the headers displayed by nedmail to the most "useful"
> ones:Date, From, To, CC (is set) and Subject.
> This helps with some mailing lists or mail providers adding loads of noisy headers.
> This only applies to the 'p' command and all headers are still visible
> when using the 'P' (display raw) command so nothing is lost.
> 
> cheers
> --phil
> 
> 
> diff 081b58a69e5efc2bbf49ee31099372527edb3fa7 uncommitted
> --- a/sys/src/cmd/upas/ned/nedmail.c
> +++ b/sys/src/cmd/upas/ned/nedmail.c
> @@ -1664,6 +1664,18 @@
>  	return c;
>  }
>  
> +void
> +printheaders(Message *m)
> +{
> +	Bprint(&out, "Date: %s\n", m->date);
> +	Bprint(&out, "From: %s\n", m->from);
> +	Bprint(&out, "To: %s\n", m->to);
> +	if(strlen(m->cc)>0)
> +		Bprint(&out, "CC: %s\n", m->cc);
> +	Bprint(&out, "Subject: %s\n", m->subject);
> +	Bprint(&out, "\n");
> +}
> +
>  Message*
>  pcmd0(Cmd *c, Message *m, int mayplumb, char *tfmt)
>  {
> @@ -1684,10 +1696,9 @@
>  	}
>  	if(m->parent == &top){
>  		seen(m);
> -		printpart(m, "unixheader", nil);
> +		/* only print headers for top part */
> +		printheaders(m);
>  	}
> -	if(printpart(m, "header", nil) > 0)
> -		Bprint(&out, "\n");
>  	cp = findctype(m);
>  	if(cp->flag & Display){
>  		if(strcmp(m->type, "text/html") == 0)
>