On Tue, Jul 29, 2008 at 12:08:41PM +0200, Ralf Schlatterbeck wrote:
> Attached please find my stack-trace -- looks like there is a problem
> with the presence after a log-off/log-on sequence (we're using p2p presence).
> The problem is reproduceable under windows and linux, just go offline and
> online again with the same account.
> #14 0x0861009c in QtTimer::updatePresence (this=0x8db7860)
>     at 
> /nobackup/checkout/kvats-2.2/wengophone/src/presentation/qt/QtTimer.cpp:52
> #15 0x08610288 in QtTimer::updatePresenceSlot (this=0x8db7860)
>     at 
> /nobackup/checkout/kvats-2.2/wengophone/src/presentation/qt/QtTimer.cpp:72
> #16 0x085d9819 in QtTimer::qt_metacall (this=0x8db7860, 
>     _c=QMetaObject::InvokeMetaMethod, _id=0, _a=0xbff0408c)
>     at 
> /nobackup/checkout/kvats-2.2/build/wengophone/src/presentation/qt/moc_QtTimer.cxx:66

OK, this dies in a method we introduced which is not in qutecom (wearing
brown paperbag now). Sorry for that.

We implemented presence-update as a Qt-Timer in the GUI (yes this *is* a
bad hack). Nevertheless, we reused code from 
wengophone/src/presentation/qt/QtIdle
which is used for detecting idle status for im presence, it's used in
wengophone/src/presentation/qt/QtWengoPhone.cpp:

        //FIXME: QtIdle must not use UserProfile but CUserProfile
        _qtIdle = new QtIdle(cUserProfile->getUserProfile(), _wengoPhoneWindow);

both methods access the model via the gui which is bad,
wengophone/src/control/profile/CUserProfile.h has:

        /**
         * Gets the UserProfile.
         *
         * TODO: This method should not exist (because it allows the GUI to 
access
         * the model directly) and so should be removed ASAP.
         */
        UserProfile & getUserProfile() {
                return _userProfile;
        }

So maybe there are more stability issues lurking here?


Concerning Presence: The best way to fix presence update would be in the osip
lib -- the currently used version doesn't fully implement presence updates.

Are there plans to move to a newer version of osip / exosip? This might render
our bad patch obsolete...

Ralf
-- 
Dr. Ralf Schlatterbeck                  Tel:   +43/2243/26465-16
Open Source Consulting                  Fax:   +43/2243/26465-23
Reichergasse 131                        www:   http://www.runtux.com
A-3411 Weidling                         email: [EMAIL PROTECTED]
osAlliance member                       email: [EMAIL PROTECTED]
_______________________________________________
QuteCom-dev mailing list
[email protected]
http://lists.qutecom.org/mailman/listinfo/qutecom-dev

Reply via email to