-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120920/
-----------------------------------------------------------
(Updated Oct. 31, 2014, 5:50 p.m.)
Review request for KDE Frameworks, KDE Graphics, KSnapshot, Aaron J. Seigo, and
Dominik Haumann.
Bugs: 312495
http://bugs.kde.org/show_bug.cgi?id=312495
Repository: ksnapshot
Description
-------
Hi, I also feel slightly impaired by Bug 312495 - Very slow "Send to" menu. So
here is a fix.
It consists of the following commits:
1. move long-running retrieval of SendTo items to separate method
2. fillSendToActionsCache at startup but not yet async
3. move code related to SendTo actions to own class and file
(this includes the kipi code but KIPI_FOUND was not set in my dev env
so thats not tested)
4. SendTo menu is built on a separate thread to speed up populating
5. delete actions in dtor
The SendTo actions are now created in QThread which is started on startup of
ksnapshot.
If the user opens the SendTo menu too soon there is a message (as single menu
item) that says
that the menu is still being constructed and that he/she should try a moment
later.
Diffs
-----
CMakeLists.txt 7ec17867e599fb8ea06b709e22fb7437e0207e9b
ksnapshot.h 898e313cf963a132387e9e5ecf422f94a1437d0a
ksnapshot.cpp 81ecd392ec7fe4076d68d0ef52ca82b6315bf150
ksnapshotsendtoactions.h PRE-CREATION
ksnapshotsendtoactions.cpp PRE-CREATION
Diff: https://git.reviewboard.kde.org/r/120920/diff/
Testing
-------
1. Start ksnapshot (notice that is as fast as before)
2. Immediately open SendTo menu => there is a message that the menu is not
ready yet.
3. Try again after 1 second: menu is populated.
4. Close ksnapshot.
1. Start ksnapshot. Wait two seconds and then click SendTo menu
=> menu is populated right away
2. Close it.
1. Start ksnapshot. Hit the SendTo menu with insanely high button click speed.
=> No crash observed.
Could NOT test the moved kipi code because KIPI_FOUND is not set on my
environment.
Thanks,
Gregor Mi
_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
