Changeset 1962ac1 for protocols

Timestamp:

2007-07-02T21:07:43Z (17 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

5d7dc00

Parents:

c377417

Message:

Fixed nick hint function to only set the nick if it's different from
the current one (otherwise the dedupe function will dedupe the nick
against itself).

File:

• protocols/nogaim.c (modified) (1 diff)

protocols/nogaim.c

@@ -471 +471 @@
                         nick_lc( newnick );
                 
-                nick_dedupe( ic->acc, handle, newnick );
-                
-                /* u->nick will be freed halfway the process, so it can't be
-                   passed as an argument. */
-                orig_nick = g_strdup( u->nick );
-                user_rename( ic->irc, orig_nick, newnick );
-                g_free( orig_nick );
+                if( strcmp( u->nick, newnick ) != 0 )
+                {
+                        /* Only do this if newnick is different from the current one.
+                           If rejoining a channel, maybe we got this nick already
+                           (and dedupe would only add an underscore. */
+                        nick_dedupe( ic->acc, handle, newnick );
+                        
+                        /* u->nick will be freed halfway the process, so it can't be
+                           passed as an argument. */
+                        orig_nick = g_strdup( u->nick );
+                        user_rename( ic->irc, orig_nick, newnick );
+                        g_free( orig_nick );
+                }
         }
 }