> On March 21, 2011, 2:45 p.m., Aaron J. Seigo wrote: > > runners/kdevelopsessions/kdevelopsessions.cpp, lines 46-56 > > <http://git.reviewboard.kde.org/r/100903/diff/1/?file=12580#file12580line46> > > > > to prevent krunner from waking up all the time, this should go into a > > slot connected to the prepare() signal; and then in a slot connected to > > teardown() delete the dirwatch.
Hm, not sure. The DirWatch only wakes up when kdevelop writes to the session files, which is "not that often", and given that the system is running at that point anyway, it won't cause unnecessary wakeups. When doing loadSessions() after krunner is pulled up, this touches disk and adds latency to krunner being available, so I'm not sure that's what we really want. (I can imagine delaying the first loading of sessions to the first teardown() appearance, though, to not unnecessarily touch disk on startup.) I've tried putting the new KDirWatch after prepare() and deleted it after teardown() signals, but got crashes in KDirWatch::addDir() after it's deleted once. I assume threading havoc here. - Sebastian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100903/#review2080 ----------------------------------------------------------- On March 21, 2011, 12:51 p.m., Sebastian Kügler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100903/ > ----------------------------------------------------------- > > (Updated March 21, 2011, 12:51 p.m.) > > > Review request for Plasma. > > > Summary > ------- > > New KRunner plugin listing kdevelop sessions. Based on the same code for > kate, only changed the reading of the sessions. > > > Diffs > ----- > > runners/CMakeLists.txt 7bcb05f > runners/kdevelopsessions/CMakeLists.txt PRE-CREATION > runners/kdevelopsessions/Messages.sh PRE-CREATION > runners/kdevelopsessions/README PRE-CREATION > runners/kdevelopsessions/kdevelopsessions.cpp PRE-CREATION > runners/kdevelopsessions/kdevelopsessions.desktop PRE-CREATION > runners/kdevelopsessions/kdevelopsessions.h PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/100903/diff > > > Testing > ------- > > Started different sessions of kdevelop from KRunner, works just fine. > > > Thanks, > > Sebastian > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
