chinmoyr updated this revision to Diff 17782.
chinmoyr added a comment.

  - added 'return'
  - fixed some typos
  - dest_file.remove() -> QFile::remove(dest). because when working with fd, 
dest_file.remove() fails but doesn't set errno to EACCES or EPERM.
  - dest_file.setPermissions() -> QFile::setPermissions(...), for the same 
reason.

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D6830?vs=17018&id=17782

BRANCH
  third

REVISION DETAIL
  https://phabricator.kde.org/D6830

AFFECTED FILES
  src/ioslaves/file/file_unix.cpp

To: chinmoyr, dfaure, #frameworks
Cc: elvisangelaccio, #frameworks

Reply via email to