Changeset 17 for psi/trunk/iris/xmpp-im/client.cpp

Timestamp:

Sep 5, 2006, 1:23:19 AM (19 years ago)

Author:

dmik

Message:

iris/xmpp-im: Added todo regarding the 100% CPU load when discovering services on some jabber servers.

File:

• psi/trunk/iris/xmpp-im/client.cpp (modified) (1 diff)

psi/trunk/iris/xmpp-im/client.cpp

@@ -472 +472 @@
 void Client::streamReadyRead()
 {
-        // HACK HACK HACK
+        /// @todo (dmik) when doing service discovery on some servers
+        //  (i.e. jabber.ru), this method's execution time can exceed up to
+        //  2,5 sec (on my Pentium D 820) that is obviously too much because
+        //  gives quite a noticeable GUI thread freeze (so that the Psi app
+        //  doesn't respond to system/input messages). Approx. 0.8 sec is taken by
+        //  s.toString(), and the rest is by QDomElement x = oldStyleNS(s.element()).
+        //  and distribute(). Ideally, parsing and distribution of lagre pieces of
+        //  data should be somehow done on a separate thread.
+        
+        // HACK HACK HACK
         QGuardedPtr<ClientStream> pstream = d->stream;