Thread
:
[ANY] Smart Link Remover
View Single Post
Agent Wesker
Senior Member
Join Date: Apr 2012
04-04-2017 , 16:29 Re: [ANY] Smart Link Remover
#
11
Quote:
Originally Posted by
Totenfluch
I have recoreded 73.000 player names and none of them had http in their name. So you can scrap that~
the MAX_NAME_LENGTH makes it literally impossible to have two links in a name that's why I didn't implement it
Other than that your regex looks extremly solid with 0 false positives and all matches in my ~350 name test - Well done!
If you send me a pull requests I'll merge it
we're having a discussion here:
https://github.com/Totenfluch/SmartLinkRemover/pull/1
if you want to join
I made a separate pull request because of the additional changes.
Also I have not yet tested on a live server so don't know if its working.
Agent Wesker
View Public Profile
Send a private message to Agent Wesker
Find More Posts by Agent Wesker