ffap versus url with "*" [patch]

ffap versus url with "*" [patch]

[Kevin Ryde]

[-- Attachment #1: Type: text/plain, Size: 821 bytes --]

I noticed with point at the start of a line like the following, M-x
ffap doesn't recognise the "*" in the url,

http://au.rd.yahoo.com/finance/news/rss/reuterau/*http://au.biz.yahoo.com/041222/19/2k95.html

it highlights and prompts with only

    http://au.rd.yahoo.com/finance/news/rss/reuterau/

"*" is uncommon, but valid.  This example is a link in a yahoo rss
feed.  The patch below adds "*" to the char matching.


Also legal but not matched are "!'()", but I'm not sure if it's a good
idea to have those.  They'd be pretty unusual, and maybe should be
treated as more likely to be surrounding punctuation, ie. left as they
are.


2004-12-22  Kevin Ryde  <user42@zip.com.au>

        * ffap.el (ffap-string-at-point-mode-alist): Add "*" to url chars,
        it can appear unencoded and has been seen from yahoo.


[-- Attachment #2: ffap.el.url-star.diff --]
[-- Type: text/plain, Size: 771 bytes --]

*** ffap.el.~1.46.~	Tue Dec 14 09:36:42 2004
--- ffap.el	Wed Dec 22 19:12:42 2004
***************
*** 962,968 ****
      ;; * no commas (good for latex)
      (file "--:$+<>@-Z_a-z~*?" "<@" "@>;.,!:")
      ;; An url, or maybe a email/news message-id:
!     (url "--:=&?$+@-Z_a-z~#,%;" "^A-Za-z0-9" ":;.,!?")
      ;; Find a string that does *not* contain a colon:
      (nocolon "--9$+<>@-Z_a-z~" "<@" "@>;.,!?")
      ;; A machine:
--- 962,968 ----
      ;; * no commas (good for latex)
      (file "--:$+<>@-Z_a-z~*?" "<@" "@>;.,!:")
      ;; An url, or maybe a email/news message-id:
!     (url "--:=&?$+@-Z_a-z~#,%;*" "^A-Za-z0-9" ":;.,!?")
      ;; Find a string that does *not* contain a colon:
      (nocolon "--9$+<>@-Z_a-z~" "<@" "@>;.,!?")
      ;; A machine:

[-- Attachment #3: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

Re: ffap versus url with "*" [patch]

[Richard Stallman]

I'll install that change in a few days, unless someone else does it
first.  Thanks.