[sup-talk] [PATCH] first pass at a simple interface to extract URIs from a message

William Morgan wmorgan-sup at masanjin.net
Sat Feb 2 21:56:40 EST 2008


Reformatted excerpts from brabuhr's message of 2008-01-29:
> This adds a new key binding to the thread-view mode (L) to assist with
> loading URIs from the body of the message.  This first pass will pull
> the URIs from the message text and loop through them while asking if
> that URI should be loaded.  Currently, there is no code to actually
> perform the load.

Looks good. I have a slight preference for a "if uris.size == 0" block
that flashes a message and returns, so that the meat of the method
doesn't have to be too indented.

My limited research indicates that there isn't a standard way (like
mailcap) of viewing a URL. So probably a "view-url" hook is the best way
to approach the actual viewing code.

-- 
William <wmorgan-sup at masanjin.net>


More information about the sup-talk mailing list