D14890: Remove QSaveFile in favor of plain old file saving

Christoph Cullmann noreply at phabricator.kde.org
Sat Aug 18 10:35:57 BST 2018


cullmann added a comment.


  Atm reverted the last addition of
  
    // did save work?
  
  - if (!saveFile->errorString().isEmpty()) {
  - BUFFER_DEBUG << "Saving file " << filename << "failed with error" << saveFile->errorString();
  - return false;
  - }
  
  +    //FIXME if (!saveFile->errorString().isEmpty()) {
  +    //    BUFFER_DEBUG << "Saving file " << filename << "failed with error" << saveFile->errorString();
  +    //    return false;
  +    //}

REPOSITORY
  R39 KTextEditor

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

To: cullmann, dhaumann, dfaure
Cc: dfaure, kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180818/2d523c94/attachment.html>


More information about the KWrite-Devel mailing list