Review Request 126113: make event filters static to decrease installed filters on qApp

David Edmundson david at davidedmundson.co.uk
Thu Nov 19 17:38:51 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126113/#review88619
-----------------------------------------------------------



src/declarativeimports/core/units.cpp (line 41)
<https://git.reviewboard.kde.org/r/126113/#comment60747>

    Why not just make this the static object...rather than having the SharedAppFilterStatic class which will have one instance of this?



src/declarativeimports/core/units.cpp (line 53)
<https://git.reviewboard.kde.org/r/126113/#comment60749>

    You only want ApplicationFontChange.
    
    qapplication.cpp:1671
    
    QEvent e(QEvent::ApplicationFontChange);        QApplication::sendEvent(QApplication::instance(), &e);



src/plasma/svg.cpp (line 47)
<https://git.reviewboard.kde.org/r/126113/#comment60748>

    An SVG with colours will have a Theme object (via SvgPrivate::cacheAndColorsTheme())
    
    ThemePrivate already has an (effecively shared) event filter on colours changing, can we use directly?


- David Edmundson


On Nov. 19, 2015, 12:08 p.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126113/
> -----------------------------------------------------------
> 
> (Updated Nov. 19, 2015, 12:08 p.m.)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Bugs: 351923
>     http://bugs.kde.org/show_bug.cgi?id=351923
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> since seems there are so many event filters installed on the QApplication installed that may give performance issues, try to use separate, singleton watchers for them to decrease the amount of eventfilters called
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/core/units.h fa2256e 
>   src/declarativeimports/core/units.cpp 4e2adae 
>   src/plasma/private/svg_p.h 597465e 
>   src/plasma/svg.cpp bcceaf7 
> 
> Diff: https://git.reviewboard.kde.org/r/126113/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151119/f753d755/attachment.html>


More information about the Plasma-devel mailing list