diff options
| author | brain <brain@e03df62e-2008-0410-955e-edbf42e46eb7> | 2006-03-08 23:16:18 +0000 |
|---|---|---|
| committer | brain <brain@e03df62e-2008-0410-955e-edbf42e46eb7> | 2006-03-08 23:16:18 +0000 |
| commit | aaf5226111f515f4baa68e95ea6a1db740828ac3 (patch) | |
| tree | ab8b622f04ebf09448ee0f2e02f32496812db003 /src/helperfuncs.cpp | |
| parent | ec97c162f84a3979a0f6a1c3443366784062ac9e (diff) | |
Removed has_channel(userrec*,chanrec*), the new preferred way of doing it is channel->HasUser(userrec)
Yeah its the other way around to the old way, but somehow, seems less backwards to me (its also faster)
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@3560 e03df62e-2008-0410-955e-edbf42e46eb7
Diffstat (limited to 'src/helperfuncs.cpp')
| -rw-r--r-- | src/helperfuncs.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/helperfuncs.cpp b/src/helperfuncs.cpp index c313b5456..2a929f3b2 100644 --- a/src/helperfuncs.cpp +++ b/src/helperfuncs.cpp @@ -1348,9 +1348,14 @@ void userlist(userrec *user,chanrec *c) CUList *ulist= c->GetUsers(); + /* Improvement by Brain - this doesnt change in value, so why was it inside + * the loop? + */ + bool has_user = c->HasUser(user,c); + for (CUList::iterator i = ulist->begin(); i != ulist->end(); i++) { - if ((!has_channel(user,c)) && (strchr(i->second->modes,'i'))) + if ((!n) && (strchr(i->second->modes,'i'))) { /* * user is +i, and source not on the channel, does not show |