[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdenetwork/kopete/protocols/oscar
From:       Stefan Gehn <sgehn () gmx ! net>
Date:       2003-11-16 12:50:41
[Download RAW message or body]

CVS commit by metz: 

Make Olivier happy by fixing uninitialized value, maybe that even fixes the idle \
weirdness I stumbled across. Again, thanks to Olivier for spotting this one


  M +6 -8      oscarcontact.cpp   1.133


--- kdenetwork/kopete/protocols/oscar/oscarcontact.cpp  #1.132:1.133
@@ -82,4 +82,5 @@ OscarContact::OscarContact(const QString
         mInfo.capabilities = 0;
         mInfo.icqextstatus = ICQ_STATUS_OFFLINE;
+        mInfo.idletime = 0;
 
         initSignals();
@@ -394,12 +395,13 @@ void OscarContact::syncGroups()
         {
                 kdDebug(14150) << k_funcinfo <<
-                        "Could not get current Oscar group for contact '" << \
                displayName() <<
-                        "'" << endl;
+                        "Could not get current Oscar group for contact '" << \
displayName() << "'" << endl;  return;
         }
 
+        /*
         kdDebug(14150) << k_funcinfo <<
                 "Current OSCAR group id=" << mGroupId <<
                 ", Current OSCAR group name='" << currentOscarGroup->name() << "'" \
<< endl; +        */
 
         // Compare the two names, to see if they're actually different
@@ -535,11 +537,7 @@ void OscarContact::slotParseUserInfo(con
                 return;
 
-        if (mInfo.idletime != u.idletime)
+        if(mInfo.idletime != u.idletime)
         {
-                setIdleTime(u.idletime*60);
-                /*if(mIdle > 0)
-                        setIdleState(Idle);
-                else // we are not idling anymore
-                        setIdleState(Active);*/
+                setIdleTime(u.idletime * 60);
                 if(u.idletime == 0)
                         emit idleStateChanged(this);


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic