Re: Prop: make URL's clickable

[Carlos Morgado <chbm chbm nu>]

On Sun, May 13, 2001 at 10:56:23AM -0700, Julian M Catchen wrote:
> I have tried out the URL patch and I like it.  It works well. 
> 
> There is a small problem that I noticed.  You used the following regex to
> match the URLs: 
> 
> ((ht|f)tps?://[^[:blank:]\r\n]+).  
> 
> Unfortunatley, if there is a ")" or "]" at the end of the url, it gets
> matched in the regex and the HTML page fails to be found because of the
> mangled URL.  So, to fix this you should add brackets and parantheses to
> regex, like this:
> 
> "((ht|f)tps?://[^[:blank:]\)\]\r\n]+)".  
> 
> Maybe consider adding all of the URL-illegal characters to this regex.
> 

iirc http://foo/file) is a legal url. that's a problem :\


-- 
Carlos Morgado - chbm(at)chbm(dot)nu - http://chbm.nu/ -- gpgkey: 0x1FC57F0A 
http://wwwkeys.pgp.net/ FP:0A27 35D3 C448 3641 0573 6876 2A37 4BB2 1FC5 7F0A
(This is a shareware sig. If you wish to receive the FULL COMERCIAL EDITION
please send me lots of money)