View Single Post
Agent Wesker
Senior Member
Join Date: Apr 2012
Old 04-04-2017 , 16:29   Re: [ANY] Smart Link Remover
Reply With Quote #11

Quote:
Originally Posted by Totenfluch View Post
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 is offline