<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/129318/">https://git.reviewboard.kde.org/r/129318/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma and Eike Hein.</div>
<div>By Jan Grulich.</div>
<p style="color: grey;"><i>Updated Lis. 3, 2016, 9:21 dop.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Remove trailing spaces, somehow this auto-remove trailing spaces was disabled in my KDevelop.</p></pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-desktop
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">In Fedora 25 there have been many improvements regarding dual graphics support and they recently added an option to Gnome to launch the application using a dicrete graphics card, by just overriding DRI_PRIME environment variable, see http://www.hadess.net/2016/10/dual-gpu-integration-in-gnome.html. This option is available based on simple dbus service implemented here https://github.com/hadess/switcheroo-control/ providing just a single property whether the user has dual GPU. This dbus thingie is enabled in Fedora 25 by default. This patch tries to add this option the same way as Gnome does. I'm just not sure about using qputenv(), but I couldn't find a better way as using QProcess, which allows to set environment variables, won't have that nice mouse effect of opening application, at least I think it won't.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>applets/kicker/plugin/actionlist.h <span style="color: grey">(869354c)</span></li>
<li>applets/kicker/plugin/actionlist.cpp <span style="color: grey">(2aa97d9)</span></li>
<li>applets/kicker/plugin/appentry.cpp <span style="color: grey">(80290e1)</span></li>
<li>applets/kicker/plugin/recentusagemodel.cpp <span style="color: grey">(24874ff)</span></li>
<li>applets/kicker/plugin/runnermatchesmodel.cpp <span style="color: grey">(4e4845c)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/129318/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>