<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/114574/">https://git.reviewboard.kde.org/r/114574/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Thanks for the patch. Unfortunately I don't think this patch has a place in KTp. I can see many scenarios where people's laptops won't suspend unexpectedly, possibly burning down in their bags while running for the bus, resulting in angry (KDE) users. If you want to disable suspending/power management, there's already a checkbox for that in the battery plasmoid, which does precisely this. I don't want to have duplicated functionality around the system hidden in a user application that virtually all users would never think of being the cause of their broken suspend. Sorry.
I hate to trash the code and work that went into it; it helps if you talk to us about your ideas first :)</pre>
<br />
<p>- Martin Klapetek</p>
<br />
<p>On March 23rd, 2014, 9:28 a.m. CET, James Smith wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy and Àlex Fiestas.</div>
<div>By James Smith.</div>
<p style="color: grey;"><i>Updated March 23, 2014, 9:28 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
ktp-kded-module
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">KeepAwake plugin inhibits suspend for certain IM states. Also adds infrastructure for presence-responsive plugins.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Compile, runtime.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>CMakeLists.txt <span style="color: grey">(d08606b)</span></li>
<li>config/telepathy-kded-config.cpp <span style="color: grey">(baaceff)</span></li>
<li>config/telepathy-kded-config.h <span style="color: grey">(0400626)</span></li>
<li>config/telepathy-kded-config.ui <span style="color: grey">(18f06f3)</span></li>
<li>keepawake.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>keepawake.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>telepathy-module.h <span style="color: grey">(2213cdf)</span></li>
<li>telepathy-module.cpp <span style="color: grey">(030a0d9)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/114574/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>