[PATCH 1/2] test: emacs mailto: URI handling

Ethan Glasser-Camp ethan.glasser.camp at gmail.com
Mon Oct 15 10:12:04 PDT 2012


Jameson Graef Rollins <jrollins at finestructure.net> writes:

> This adds a test for proposed rfc6068 "mailto:" URI handling.  The
> proposed function would be called 'notmuch-mua-mailto'.  The test
> provides an example mailto: string that should test some subset of the
> rfc6068 specification: http://tools.ietf.org/html/rfc6068

Hi! Just going through the patch queue. This test, at least, looks
sane, so why not apply it?

As for the mailto: handler itself, you wrote in
id:"87vcin2fo6.fsf at servo.finestructure.net" that you would add something
to the next version, thereby implying that there would be a next
version. Do you still intend to release such a new version soon? I would
love to review it. This version looks fine to me, with one minor concern.

> +	   ;; FIXME: need to decode all html encodings in uri.
> +	   (mailto (replace-regexp-in-string "&" "&" mailto))

This is odd. Shouldn't the navigator be decoding all that stuff while
parsing, before you even get to it? How often do you get mailto links
with & in them? You obviously wouldn't have put it in there if it
weren't necessary.

I would be +1 on merging this patch or the next version.

Ethan


More information about the notmuch mailing list