<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="http://git.reviewboard.kde.org/r/111086/">http://git.reviewboard.kde.org/r/111086/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On June 18th, 2013, 1:48 p.m. UTC, <b>Kevin Ottens</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ship It!</pre>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hold on... Per Fredrik Höglund email which I totally overlooked until now, we're clearly doing that one wrong.
Please rework this patch so that kwindowsystem is untouched but the kdeui copy of kxerrorhandler goes in kde4support instead.

Sorry for the troubles, thanks in advance.</pre>
<br />










<p>- Kevin</p>


<br />
<p>On June 18th, 2013, 10:51 a.m. UTC, Benjamin Port wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 Benjamin Port.</div>


<p style="color: grey;"><i>Updated June 18, 2013, 10:51 a.m.</i></p>






<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;">Finish to moving KXErrorHandler

Export and rename kxerrorhandler_p.h
Move kexerrorhandler test</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>kdeui/CMakeLists.txt <span style="color: grey">(c05adbd)</span></li>

 <li>kdeui/tests/CMakeLists.txt <span style="color: grey">(dfe6292)</span></li>

 <li>kdeui/tests/kxerrorhandlertest.cpp <span style="color: grey">(8b624f2)</span></li>

 <li>kdeui/util/kxerrorhandler.h <span style="color: grey">(be5cb76)</span></li>

 <li>kdeui/util/kxerrorhandler.cpp <span style="color: grey">(48d8abe)</span></li>

 <li>tier1/kwindowsystem/CMakeLists.txt <span style="color: grey">(3032ac7)</span></li>

 <li>tier1/kwindowsystem/src/CMakeLists.txt <span style="color: grey">(b191830)</span></li>

 <li>tier1/kwindowsystem/src/kwindowinfo_x11.cpp <span style="color: grey">(f382e9c)</span></li>

 <li>tier1/kwindowsystem/src/kwindowsystem_x11.cpp <span style="color: grey">(cc2c5a0)</span></li>

 <li>tier1/kwindowsystem/src/kxerrorhandler.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tier1/kwindowsystem/src/kxerrorhandler.cpp <span style="color: grey">(67062de)</span></li>

 <li>tier1/kwindowsystem/src/kxerrorhandler_p.h <span style="color: grey">(53237a3)</span></li>

 <li>tier1/kwindowsystem/src/kxutils.cpp <span style="color: grey">(33bd678)</span></li>

 <li>tier1/kwindowsystem/tests/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>

 <li>tier1/kwindowsystem/tests/kxerrorhandlertest.cpp <span style="color: grey">(PRE-CREATION)</span></li>

</ul>

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







  </td>
 </tr>
</table>








  </div>
 </body>
</html>