[PATCH v1 1/1] emacs: Easy access to URLs mentioned in the current message

David Bremner david at tethera.net
Thu Mar 28 06:20:03 PDT 2019


David Edmondson <dme at dme.org> writes:

> On Thursday, 2019-03-28 at 08:23:37 -03, David Bremner wrote:
>
>> David Edmondson <dme at 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?
>
> Yes.
>
> Do you want an updated patch or will you fix it on the fly?
>

I can fix it.

d


More information about the notmuch mailing list