Well, one slight problem is the \b inside brackets which means a backslash
in CFMX, not a word boundary. I'm using [^[:space:]] instead now and it
seems to work perfectly. And no, I don't want to use \s rather than
[[:space:]] as [[:space:]] is easier to read/understand (self documentation)
and I don't see any speed difference between them. If there is a difference
in ability/speed/performance/load, please let me know and I'll change.
Thanks

REReplace(text, '\b(https?://[^[:space:]]+)\b', '<A HREF="\1"
target="_blank">\1</A>', 'all')


> I'm adding some code to the list archives to take any link posted and turn
> it into a 'real' link. This is the code I'm using:
> REReplace(text, '\b(https?://[^\b ]+)\b', '<A HREF="\1"
> target="_blank">\1</A>', 'all')
> 
> Does anyone see any problems with it. Any place it could fail? Any
> suggestions?
> Thanks
> 
> 

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~|
Special thanks to the CF Community Suite Silver Sponsor - CFDynamics
http://www.cfdynamics.com

Message: http://www.houseoffusion.com/lists.cfm/link=i:21:790
Archives: http://www.houseoffusion.com/cf_lists/threads.cfm/21
Subscription: http://www.houseoffusion.com/lists.cfm/link=s:21
Unsubscribe: 
http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=11502.10531.21
Donations & Support: http://www.houseoffusion.com/tiny.cfm/54

Reply via email to