On Thursday, 2019-03-28 at 08:23:37 -03, David Bremner wrote: > David Edmondson <[email protected]> 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? dme. -- I'm catching up with myself! _______________________________________________ notmuch mailing list [email protected] https://notmuchmail.org/mailman/listinfo/notmuch
