David Edmondson <dme@dme.org> writes:


> +(defun notmuch-show--browse-urls ()
> +  "Offer to browse any URLs in the current message."
> +  (interactive)
> +  (let ((urls (notmuch-show--gather-urls)))
> +    (if urls
> +	(browse-url (completing-read "Browse URL: " (cdr urls) nil nil (car urls)))
> +      (message "No URLs found."))))
> +
>  (provide 'notmuch-show)

It's a minor point, but should this really have -- to signal a private
function? It's bound to key which sortof makes it defacto public, no?

d


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