Gnus development mailing list
 help / color / mirror / Atom feed
From: asjo@koldfront.dk (Adam Sjøgren)
To: ding@gnus.org
Subject: Re: message-citation-line-format - should %F have a fall back?
Date: Thu, 26 Feb 2015 23:38:29 +0100	[thread overview]
Message-ID: <87oaog70yi.fsf@topper.koldfront.dk> (raw)
In-Reply-To: <b4m4mq9pkrm.fsf@jpl.org>

Katsumi Yamaoka writes:

> On Wed, 25 Feb 2015 21:34:24 +0100, Adam Sjøgren wrote:
>> Should %F in message-citation-line-format fall back to something
>> different than nil, if no first name can be detected?

> The most corner‐cutting way would be:

> --- message.el~	2015-01-29 02:00:24.303787500 +0000
> +++ message.el	2015-02-26 00:40:28.328441100 +0000
> @@ -4085,3 +4085,3 @@
>  	      (push ?E lst) (push "<E>" lst)
> -	      (push ?F lst) (push fname lst)
> +	      (push ?F lst) (push (or fname name-or-net) lst)
>  	      ;; We might want to use "" instead of "<X>" later.
>
> This is what %N does.

I was testing this:

--- a/lisp/message.el
+++ b/lisp/message.el
@@ -4036,7 +4036,7 @@ See `message-citation-line-format'."
 			      from)
 		   (error nil)))
 	   (name (car data))
-	   (fname name)
+	   (fname-or-net (or name (car (cdr data)) from))
 	   (lname name)
 	   (net (car (cdr data)))
 	   (name-or-net (or (car data)
@@ -4083,7 +4083,7 @@ See `message-citation-line-format'."
                       (setq fname lname lname newlname)))))
 	      ;; The following letters are not used in `format-time-string':
 	      (push ?E lst) (push "<E>" lst)
-	      (push ?F lst) (push fname lst)
+	      (push ?F lst) (push fname-or-net lst)
 	      ;; We might want to use "" instead of "<X>" later.
 	      (push ?J lst) (push "<J>" lst)
 	      (push ?K lst) (push "<K>" lst)

I don't know if that is better or worse...


  Best regards,

    Adam

-- 
 "Apparantly I was misinformed."                              Adam Sjøgren
                                                         asjo@koldfront.dk




      reply	other threads:[~2015-02-26 22:38 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-25 20:34 Adam Sjøgren
2015-02-26  0:41 ` Katsumi Yamaoka
2015-02-26 22:38   ` Adam Sjøgren [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87oaog70yi.fsf@topper.koldfront.dk \
    --to=asjo@koldfront.dk \
    --cc=ding@gnus.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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 NNTP newsgroup(s).