Review Request: KRunner: Only one simultanous timer event

Aaron Seigo aseigo at kde.org
Sun Nov 14 20:24:03 CET 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5844/#review8722
-----------------------------------------------------------

Ship it!


good catch.


/trunk/KDE/kdebase/workspace/krunner/krunnerdialog.h
<http://svn.reviewboard.kde.org/r/5844/#comment9271>

    may as well keep with the bools above and explicitly set it to be part of the bit field with ": 1;"


- Aaron


On 2010-11-14 11:20:53, Matthias Fuchs wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5844/
> -----------------------------------------------------------
> 
> (Updated 2010-11-14 11:20:53)
> 
> 
> Review request for Plasma and Aaron Seigo.
> 
> 
> Summary
> -------
> 
> This patch makes that there is only one timer at once looking for the mouse-cursor position.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/krunner/krunnerdialog.h 1196906 
>   /trunk/KDE/kdebase/workspace/krunner/krunnerdialog.cpp 1196906 
> 
> Diff: http://svn.reviewboard.kde.org/r/5844/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Matthias
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20101114/15f57543/attachment-0001.htm 


More information about the Plasma-devel mailing list