bug-gnu-emacs
[Top][All Lists]
Advanced

[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 <address@hidden> 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/address@hidden
> - "[" 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.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]