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

List:       nepomuk
Subject:    Re: [Nepomuk] Review Request: Fix for the rapid change in state of
From:       Sebastian Trüg <trueg () kde ! org>
Date:       2011-09-29 8:03:31
Message-ID: 4E842653.7080200 () kde ! org
[Download RAW message or body]

Right, we need a QTImer member variable.

On 09/29/2011 09:55 AM, François K. wrote:
> 
> ----- "Martin Klapetek" <martin.klapetek@gmail.com> a écrit :
> 
> > De: "Martin Klapetek" <martin.klapetek@gmail.com>
> > À: "Martin Klapetek" <martin.klapetek@gmail.com>, "Smit Shah" \
> > <who828@gmail.com>, "François Kubler" <francois@kubler.org>, "Nepomuk" \
> > <nepomuk@kde.org> Envoyé: Jeudi 29 Septembre 2011 09h47:09 GMT +01:00 Amsterdam \
> >                 / Berlin / Berne / Rome / Stockholm / Vienne
> > Objet: Re: [Nepomuk] Review Request: Fix for the rapid change in state of nepomuk \
> > system tray icon 
> > This is an automatically generated e-mail. To reply, visit:
> > http://git.reviewboard.kde.org/r/102726/
> > 
> > 
> > 
> > 
> > On September 28th, 2011, 9:52 p.m., François Kubler wrote:
> > 
> > 
> > nepomuk/controller/systray.cpp (Diff revision 2)
> > 	Nepomuk::SystemTray::SystemTray( QObject* parent )
> > 	112 	m_updateTimer . start ( 5000 ); When started like that, the
> > timeout SIGNAL will be fired every 5 seconds. I don't think this is
> > needed, is it ?
> > Shouldn't the SIGNAL be fired only once to tell the widget to blink ?
> > (if so, you could use something like setSingleShot(true) ). There's
> > also a static method for that -- QTimer::singleShot()
> > 
> > 
> > - Martin
> 
> 
> Yes, but the static method can't be cancelled (for example, if indexing is stopped \
> after 2 seconds), can it ? I'm really new to Qt, so you'd better check what I say \
> twice :) 
> Have a nice day,
> 
_______________________________________________
Nepomuk mailing list
Nepomuk@kde.org
https://mail.kde.org/mailman/listinfo/nepomuk


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

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