[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#14950: 24.3.50; gnus: Fontification of URLs is not satisfactory
From: |
Katsumi Yamaoka |
Subject: |
bug#14950: 24.3.50; gnus: Fontification of URLs is not satisfactory |
Date: |
Fri, 26 Jul 2013 08:40:56 +0900 |
User-agent: |
Gnus/5.130008 (真 Gnus v0.8) Emacs/24.3.50 (i686-pc-cygwin) |
Andreas Schwab wrote:
> Katsumi Yamaoka <yamaoka@jpl.org> writes:
>> What should be improved is the default value of the user option
>> `gnus-button-url-regexp'. But it is too complicated to let me
>> encourage to do it without enbugging. Anyone?
> How about this?
> diff --git a/lisp/gnus/gnus-art.el b/lisp/gnus/gnus-art.el
> index 31a108a..2433db2 100644
> --- a/lisp/gnus/gnus-art.el
> +++ b/lisp/gnus/gnus-art.el
> @@ -7177,7 +7177,8 @@ groups."
> "\\(?:"
> ;; Match paired parentheses, e.g. in Wikipedia URLs:
> ;; http://thread.gmane.org/47B4E3B2.3050402@gmail.com
> - "[" chars punct "]+" "(" "[" chars punct "]+" "[" chars "]*)" "["
> chars "]*"
> + "[" chars punct "]+" "(" "[" chars punct "]+" "[" chars "]*)"
> + "\\(?:" "[" chars punct "]+" "[" chars "]" "\\)?"
> "\\|"
> "[" chars punct "]+" "[" chars "]"
> "\\)"))
Great! I understood what what you changed meant. Thanks.
The original one only matches the last part of a url like:
http://.../abc.def(ghi)jklmno
But yours matches:
http://.../abc.def(ghi)jkl.mno
Maybe making it match the one like
http://.../abc.def(ghi)jkl.mno/pqr(stu)vwx
is not necessary. I've installed your patch.