<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/101702/">http://git.reviewboard.kde.org/r/101702/</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 22nd, 2011, 7 p.m., <b>David Faure</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;">OK.</pre>
 </blockquote>




 <p>On June 22nd, 2011, 7:24 p.m., <b>Sebastian Trueg</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;">Just to be clear: does this mean: "push it now" or "push it after the release of 4.7"?</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;">Hmm, true, I think things are pretty frozen right now. In theory, conversion operators is something I'm wary of, they can make code ambiguous etc.
But in this particular case, QList<QUrl> and QList<KUrl> are pretty incompatible otherwise, so I can't see any risk of trouble arising from this change. So I'm not objecting to it being committed -- but you retain all responsibility in case of a breakage :-)

Since this is just convenience you can work around, I guess it's safer to just push to master (4.7 is already branched).</pre>
<br />








<p>- David</p>


<br />
<p>On June 20th, 2011, 11:58 a.m., Sebastian Trueg wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for kdelibs.</div>
<div>By Sebastian Trueg.</div>


<p style="color: grey;"><i>Updated June 20, 2011, 11:58 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;">Internally in Nepomuk we use QUrl instead of KUrl since Soprano uses QUrl and we do not need the additional power of KUrl most of the time. Thus, conversion between KUrl and QUrl is important. This patch adds a constructor to KUrl::List which allows to use a QList<QUrl> as basis and an operator which provides automatic conversion from KUrl::List to QList<QUrl>.</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>kdecore/io/kurl.h <span style="color: grey">(52af985)</span></li>

 <li>kdecore/io/kurl.cpp <span style="color: grey">(90ececf)</span></li>

</ul>

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




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








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