spam bots nicks

[Ofloo]

If I where you I'd make it return "1" for a match and return "0" no match, and inside the proc to check the nick: if {[isSpamNick $nick]} { # is a match so ban or whatever }

Also, isn't a regexp fatser than that loop? Use break to stop a loop not return with something like this:

Code: Select all

proc isSpamNick {nick} { 
  global names 
  set blah 0
  foreach {x} $names { 
    if {[string match -nocase *${x}* $nick]} { 
      set blah 1 ; break
    } 
  } 
  return "$blah"
}

can i ask what exact advantage breaking a loop has, in this case cause where not working with I/O, so eof can't error when you do.. just wondering..

[caesar]

"I found a drone, ban it!" What's the point on continuing the loop till the end of the list? It breakes the loop and continues normaly with the proc when a match was found, or dose it need to make more than I match?