Review Request: Use window() as KNotificationItem::associatedWidget() to avoid native widgets

Marco Martin notmart at gmail.com
Fri Sep 25 11:55:06 CEST 2009


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

Ship it!


well, looking at all the code again it would just have th same effect of setAssociatedWidget(0), simply the activated signal(clicking) won't do anything by itself but forward the signal itself, you're right, i need to expand the apidox on that.

aaanyways, the patch looks good, please go for it.

- Marco


On 2009-09-24 03:20:31, Christoph Feck wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1707/
> -----------------------------------------------------------
> 
> (Updated 2009-09-24 03:20:31)
> 
> 
> Review request for kdelibs and Plasma.
> 
> 
> Summary
> -------
> 
> Use the window() of the passed parent() as the associated widget to avoid creating a native widget for the child due to all the winId() calls. Alien widgets are much faster.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/kdeui/notifications/knotificationitem.cpp 1027406 
> 
> Diff: http://reviewboard.kde.org/r/1707/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Christoph
> 
>



More information about the Plasma-devel mailing list