Changeset 3f9440d

Timestamp:

2006-06-26T21:50:12Z (18 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

bf25fa3, fef6116

Parents:

2f13222

Message:

/NAMES replies are more efficient now.

File:

• irc.c (modified) (1 diff)

irc.c

@@ -649 +649 @@
 void irc_names( irc_t *irc, char *channel )
 {
-        user_t *u = irc->users;
-        char *s;
-        int control = ( g_strcasecmp( channel, irc->channel ) == 0 );
+        user_t *u;
+        char namelist[385] = "";
         struct conversation *c = NULL;
-        
-        if( !control )
-                c = conv_findchannel( channel );
         
         /* RFCs say there is no error reply allowed on NAMES, so when the
            channel is invalid, just give an empty reply. */
         
-        if( control || c ) while( u )
-        {
-                if( u->online )
-                {
-                        if( u->gc && control )
+        if( g_strcasecmp( channel, irc->channel ) == 0 )
+        {
+                for( u = irc->users; u; u = u->next ) if( u->online )
+                {
+                        if( strlen( namelist ) + strlen( u->nick ) > sizeof( namelist ) - 4 )
                         {
-                                if( set_getint( irc, "away_devoice" ) && !u->away )
-                                        s = "+";
-                                else
-                                        s = "";
-                                
-                                irc_reply( irc, 353, "= %s :%s%s", channel, s, u->nick );
+                                irc_reply( irc, 353, "= %s :%s", channel, namelist );
+                                *namelist = 0;
                         }
-                        else if( !u->gc )
+                        
+                        if( u->gc && !u->away && set_getint( irc, "away_devoice" ) )
+                                strcat( namelist, "+" );
+                        
+                        strcat( namelist, u->nick );
+                        strcat( namelist, " " );
+                }
+        }
+        else if( ( c = conv_findchannel( channel ) ) )
+        {
+                GList *l;
+                char *ops = set_getstr( irc, "ops" );
+                
+                /* root and the user aren't in the channel userlist but should
+                   show up in /NAMES, so list them first: */
+                sprintf( namelist, "%s%s %s%s ", strcmp( ops, "root" ) == 0 || strcmp( ops, "both" ) ? "@" : "", irc->mynick,
+                                                 strcmp( ops, "user" ) == 0 || strcmp( ops, "both" ) ? "@" : "", irc->nick );
+                
+                for( l = c->in_room; l; l = l->next ) if( ( u = user_findhandle( c->gc, l->data ) ) )
+                {
+                        if( strlen( namelist ) + strlen( u->nick ) > sizeof( namelist ) - 4 )
                         {
-                                if( strcmp( u->nick, irc->mynick ) == 0 && ( strcmp( set_getstr( irc, "ops" ), "root" ) == 0 || strcmp( set_getstr( irc, "ops" ), "both" ) == 0 ) )
-                                        s = "@";
-                                else if( strcmp( u->nick, irc->nick ) == 0 && ( strcmp( set_getstr( irc, "ops" ), "user" ) == 0 || strcmp( set_getstr( irc, "ops" ), "both" ) == 0 ) )
-                                        s = "@";
-                                else
-                                        s = "";
-                                
-                                irc_reply( irc, 353, "= %s :%s%s", channel, s, u->nick );
+                                irc_reply( irc, 353, "= %s :%s", channel, namelist );
+                                *namelist = 0;
                         }
-                }
-                
-                u = u->next;
-        }
-        
-        /* For non-controlchannel channels (group conversations) only root and
-           you are listed now. Time to show the channel people: */
-        if( !control && c )
-        {
-                GList *l;
-                
-                for( l = c->in_room; l; l = l->next )
-                        if( ( u = user_findhandle( c->gc, l->data ) ) )
-                                irc_reply( irc, 353, "= %s :%s%s", channel, "", u->nick );
-        }
+                        
+                        strcat( namelist, u->nick );
+                        strcat( namelist, " " );
+                }
+        }
+        
+        if( *namelist )
+                irc_reply( irc, 353, "= %s :%s", channel, namelist );
         
         irc_reply( irc, 366, "%s :End of /NAMES list", channel );