Re: Review Request: kickoff: save recent applications list on every change to it

2012-07-04 Thread Andriy Gapon
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/ --- (Updated July 4, 2012, 7:57 a.m.) Review request for Plasma and Trever

Re: Review Request: kickoff: save recent applications list on every change to it

2012-07-04 Thread Andriy Gapon
On May 31, 2012, 10:09 a.m., Aaron J. Seigo wrote: plasma/desktop/applets/kickoff/core/recentapplications.cpp, line 42 http://git.reviewboard.kde.org/r/105112/diff/1/?file=65943#file65943line42 holding on to groups like this is a bit dangerous; you need to be able to know that

Re: Review Request: kickoff: save recent applications list on every change to it

2012-07-04 Thread Andriy Gapon
On May 31, 2012, 10:09 a.m., Aaron J. Seigo wrote: the real fubar here is that it stores this information internally in its own config file. this really ought to be stored/retrieved from nepomuk and/or zeitgeist. i've cc'd Trever on this because he may have something to say about

Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Andriy Gapon
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/ --- Review request for Plasma. Description --- Currently recent

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/ --- (Updated May 31, 2012, 10:08 a.m.) Review request for Plasma and Trever

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/#review14293 --- the real fubar here is that it stores this information

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Trever Fischer
On May 31, 2012, 10:09 a.m., Aaron J. Seigo wrote: the real fubar here is that it stores this information internally in its own config file. this really ought to be stored/retrieved from nepomuk and/or zeitgeist. i've cc'd Trever on this because he may have something to say about

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Aaron J. Seigo
On May 31, 2012, 10:09 a.m., Aaron J. Seigo wrote: the real fubar here is that it stores this information internally in its own config file. this really ought to be stored/retrieved from nepomuk and/or zeitgeist. i've cc'd Trever on this because he may have something to say about

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Mark
On Thu, May 31, 2012 at 3:36 PM, Trever Fischer tdfisc...@fedoraproject.org wrote: This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/ On May 31st, 2012, 10:09 a.m., *Aaron J. Seigo* wrote: the real fubar here is that it stores this

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Trever Fischer
On Thu, 2012-05-31 at 16:45 +0200, Mark wrote: On Thu, May 31, 2012 at 3:36 PM, Trever Fischer tdfisc...@fedoraproject.org wrote: This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105112/ On May

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Trever Fischer
On May 31, 2012, 10:09 a.m., Aaron J. Seigo wrote: the real fubar here is that it stores this information internally in its own config file. this really ought to be stored/retrieved from nepomuk and/or zeitgeist. i've cc'd Trever on this because he may have something to say about

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Ivan Čukić
On May 31, 2012, 10:09 a.m., Aaron J. Seigo wrote: the real fubar here is that it stores this information internally in its own config file. this really ought to be stored/retrieved from nepomuk and/or zeitgeist. i've cc'd Trever on this because he may have something to say about

Re: Review Request: kickoff: save recent applications list on every change to it

2012-05-31 Thread Seif Lotfy
On May 31, 2012, 10:09 a.m., Aaron J. Seigo wrote: the real fubar here is that it stores this information internally in its own config file. this really ought to be stored/retrieved from nepomuk and/or zeitgeist. i've cc'd Trever on this because he may have something to say about