Much nicer :-)

On Fri, 27 Jan 2012 09:44:26 +0100, Pieter Praet <pie...@praet.org> wrote:
> +(defcustom notmuch-show-stash-mlarchive-link-alist
> +  '(("Gmane" . "http://mid.gmane.org/";)
> +    ("MARC" . "http://marc.info/?i=";)
> +    ("Mail Archive, The" . "http://mail-archive.com/search?l=mid&q=";)
> +    ;; FIXME: can these services be searched by `Message-Id' ?
> +    ;; ("MarkMail" . "http://markmail.org/";)
> +    ;; ("Nabble" . "http://nabble.com/";)
> +    ;; ("opensubscriber" . "http://opensubscriber.com/";)
> +    )
> +  "List of Mailing List Archives to use when stashing links."
> +  :type '(alist :key-type (string :tag "Name")
> +             :value-type (string :tag "URL"))
> +  :group 'notmuch-show)

Add a comment saying that the URI is the result of concatenating this
with the message id.

If someone wants to do something more complicated they can submit
another patch (maybe it should be an option to have the `cdr' be a
function that is called with the message id as an argument?).

> +        (substring (notmuch-show-get-message-id) 4 -1))))

We should put this logic in one place and re-use it.

Attachment: pgpb0Ev8NjSJA.pgp
Description: PGP signature

_______________________________________________
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch

Reply via email to