D22105: WIP : Fix SFTP Plugin of KIO for Windows

David Faure noreply at phabricator.kde.org
Sun Jul 21 11:53:24 BST 2019


dfaure requested changes to this revision.
dfaure added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> kio_sftp.cpp:266
>  
> -        qCDebug(KIO_SFTP_LOG) << "name=" << name << " instruction=" << instruction << " prompts=" << n;
> +        qDebug() << "name=" << name << " instruction=" << instruction << " prompts=" << n;
>  

Use `kdebugsettings` to enable sftp debug output instead of qCDebug->qDebug (which adds a lot of noise to the review)

> brute4s99 wrote in kio_sftp.cpp:2050-2051
> it uses buff.st_atime . Since I'm removing use of buff, I'm not sure how to handle this. For now I've commented out setting the file access time instruction for now.

Just use partFile.fileTime(QFileInfoFileAccessTime)

> kio_sftp.cpp:2052
> +                    //                         QFileDevice::FileTime(QFileDevice::FileAccessTime));
> +                    receivedFile.setFileTime(dt, QFileDevice::FileTime(QFileDevice::FileModificationTime));
> +                }

You can remove the FileTime() type conversion, just do setFileTime(dt, QFileDevice::FileModificationTime)

REPOSITORY
  R320 KIO Extras

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

To: brute4s99, albertvaka, vonreth, sredman, sitter, dfaure
Cc: pino, andriusr, kde-frameworks-devel, kfm-devel, aprcela, fprice, LeGast00n, sbergeron, fbampaloukas, alexde, feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190721/e81f87fc/attachment.html>


More information about the Kde-frameworks-devel mailing list