<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/111724/">http://git.reviewboard.kde.org/r/111724/</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;">::rename also is only for files on the same partition, so that's not the issue. The problem is more likely the error handling, i.e. recognizing that the renaming failed because of that particular reason, and falling back (in libkio) to a copy followed by a delete (this only happens when the error code is ERROR_UNSUPPORTED_ACTION).

Anyway. I think we're making this more complex than it has to be. This is unix-only code, and it doesn't even have the largefile issues that other APIs have.

KDE_rename() is always ::rename(), on Unix, as can be seen in kde_file.h.

So I suggest to just use ::rename() instead of KDE_rename().</pre>

 <br />

<p>- David</p>

<br />

<p>On July 26th, 2013, 4:10 p.m. UTC, Martin Klapetek 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 Martin Klapetek.</div>

<p style="color: grey;"><i>Updated July 26, 2013, 4:10 p.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;">Ports KDE_rename(..) to QFile::rename(..). All tests pass except those where moving dirs between partitions occur. The Qt4 docs says "rename will fail if newName and oldName are not on the same partition" for QDir::rename(); there's no such mention in Qt 5.dev docs but it still seems to be the case and I'm unsure what to do, please advise.</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;">All tests pass except moving dirs between partitions.</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>kioslave/file/file_unix.cpp <span style="color: grey">(64cffa0)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>