Make clickable - ereg problem

[Anon]

I have a problem with this in other ways excellent code that I have managed to splice from several sources. It's purpose is to make a number of http/ftp/mailto-links in a piece of text clickable. It works well, but I have a problem. I assumes that all links end by the next space. How do I convert it render links that end on other characters corretly? These could be point(.), comma (,) and ) plus others that I have not thought of or met yet.

This is my function, which works well in 99.5% of the cases:

function make_clickable($text)
{
$text = eregi_replace("((http|mailto|ftp):\/\/[^{[:space:]<>]{1,})",} "<A HREF='\1'>\1</A>", $text);
return($text);
}

Thanks

Martin

[Anon]

$text= eregi_replace("([a-z]://[[:alnum:]#?/&=@;.,])",
"<A HREF='\1'>\1</A>", $text);

This will catch other protocols as well (telnet, gopher, etc.)

[Anon]

Steve,

Thanks! Just what I needed. The list of non-terminal (excuse the term) characters needed to be extended a bit though. I included both dash and underscore and more might appear. I will look up a list of legal URL characters somewhere. This is my current function:

$text= eregi_replace("([a-z]://[[:alnum:]#?/&=@;.,_-])", "<A HREF='\1'>\1</A>", $text);

Works like a charm.

Martin