Re: Review Request 122371: Disable session management for kwrited
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122371/ --- (Updated Feb. 9, 2015, 5:52 p.m.) Status -- This change has been marked as submitted. Review request for Plasma. Bugs: 343550 https://bugs.kde.org/show_bug.cgi?id=343550 Repository: kwrited Description --- kwrited has its autostart file IMHO doesn't need session management. Diffs - kwrited.cpp a4dac6b Diff: https://git.reviewboard.kde.org/r/122371/diff/ Testing --- no kwrited started by session. Thanks, Xuetian Weng ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122371: Disable session management for kwrited
On Feb. 8, 2015, 1:49 p.m., David Edmundson wrote: kwrited.cpp, line 74 https://git.reviewboard.kde.org/r/122371/diff/1/?file=346325#file346325line74 why this? It's fine. Since kwrited doesn't have any UI, we can safely get away with this. It will reduce loading the current style and decrease memory usage. We do the same thing in Baloo. - Vishesh --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122371/#review75601 --- On Feb. 1, 2015, 8:55 p.m., Xuetian Weng wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122371/ --- (Updated Feb. 1, 2015, 8:55 p.m.) Review request for Plasma. Bugs: 343550 https://bugs.kde.org/show_bug.cgi?id=343550 Repository: kwrited Description --- kwrited has its autostart file IMHO doesn't need session management. Diffs - kwrited.cpp a4dac6b Diff: https://git.reviewboard.kde.org/r/122371/diff/ Testing --- no kwrited started by session. Thanks, Xuetian Weng ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122371: Disable session management for kwrited
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122371/#review75599 --- bump - Xuetian Weng On Feb. 1, 2015, 8:55 p.m., Xuetian Weng wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122371/ --- (Updated Feb. 1, 2015, 8:55 p.m.) Review request for Plasma. Bugs: 343550 https://bugs.kde.org/show_bug.cgi?id=343550 Repository: kwrited Description --- kwrited has its autostart file IMHO doesn't need session management. Diffs - kwrited.cpp a4dac6b Diff: https://git.reviewboard.kde.org/r/122371/diff/ Testing --- no kwrited started by session. Thanks, Xuetian Weng ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 122371: Disable session management for kwrited
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122371/#review75601 --- Ship it! kwrited.cpp https://git.reviewboard.kde.org/r/122371/#comment52288 why this? - David Edmundson On Feb. 1, 2015, 8:55 p.m., Xuetian Weng wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122371/ --- (Updated Feb. 1, 2015, 8:55 p.m.) Review request for Plasma. Bugs: 343550 https://bugs.kde.org/show_bug.cgi?id=343550 Repository: kwrited Description --- kwrited has its autostart file IMHO doesn't need session management. Diffs - kwrited.cpp a4dac6b Diff: https://git.reviewboard.kde.org/r/122371/diff/ Testing --- no kwrited started by session. Thanks, Xuetian Weng ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel