<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/114390/">http://git.reviewboard.kde.org/r/114390/</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 d70b472678720e5637e61b12360dbe0aad1e0673 by Raphael Kubo da Costa to branch KDE/4.11.</pre>

 <br />

<p>- Commit Hook</p>

<br />

<p>On December 10th, 2013, 5:37 p.m. UTC, Raphael Kubo da Costa 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 Nepomuk, Gabriel Poesia and Vishesh Handa.</div>

<div>By Raphael Kubo da Costa.</div>

<p style="color: grey;"><i>Updated Dec. 10, 2013, 5:37 p.m.</i></p>

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

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

nepomuk-core

</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;">nepomukctl: Do not create a global static QTextStream.

Creating global static objects is a big no-no, and is properly documented in

KDE's policies. One never knows the order in which they are created and

destroyed, which was causing crashes at least on FreeBSD when libcxxrt is

used, as Qt's Q_GLOBAL_STATIC cleanup code would destroy QTextCodec

instances still referenced by that global QTextStream.

Since the QTextStream was only used as glorified fprintf(), resort to just

printing to std::cerr directly.</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;">`nepomukctl status' and other commands finally work instead of crashing.</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>tools/nepomukctl/main.cpp <span style="color: grey">(93d8a78779efb2d64ef5de6072d8d19ff09dd5fe)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>