<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/115721/">https://git.reviewboard.kde.org/r/115721/</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;">This review has been submitted with commit b0df340288405dea21a9dc6768bfe0f652ca2849 by Martin Gräßlin to branch master.</pre>

 <br />

<p>- Commit Hook</p>

<br />

<p>On February 17th, 2014, 8:15 a.m. UTC, Martin Gräßlin 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 KDE Frameworks.</div>

<div>By Martin Gräßlin.</div>

<p style="color: grey;"><i>Updated Feb. 17, 2014, 8:15 a.m.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

kio

</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;">Reduce usage of HAVE_X11 in KRun and make it usable on non-X11

The diversion for some methods by having a KStartupInfoId is removed.

As KStartupInfoId is not an X11 specific class we can work with normal

method overloading instead of needing ifdefs.

In addition the code which uses X11 specific functionality is adjusted

to be only used if we are on platform X11. For other platforms the

same fallback is used as already present through the ifdef diversion.</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;">used kruntest on X11 and Wayland - together with setting QT_QPA_PLATFORM env variable we get the expected result.</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>src/widgets/krun.cpp <span style="color: grey">(7c378f248d4f3f9bf05449918c27179aeca33cbd)</span></li>

 <li>src/widgets/krun_p.h <span style="color: grey">(05d63670c25a5c73fa6f690fdd73834167f9caf8)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/115721/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>