brute4s99 added inline comments.

INLINE COMMENTS

> dfaure wrote in kio_sftp.cpp:2037
> Why ReadWrite, if we know it doesn't exist?
> 
> Does setFileTime() even need open() first? I wouldn't have thought so.

`setFileTime()` needs the file to be open. 
Source : https://doc.qt.io/qt-5/qfiledevice.html#setFileTime

REPOSITORY
  R320 KIO Extras

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

To: brute4s99, albertvaka, vonreth, sredman, sitter, dfaure
Cc: andriusr, kde-frameworks-devel, kfm-devel, aprcela, fprice, LeGast00n, 
sbergeron, fbampaloukas, alexde, feverfew, meven, michaelh, spoorun, 
navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov

Reply via email to