D23832: Backup on save: Support time and date string replacements

Christoph Cullmann noreply at phabricator.kde.org
Wed Sep 11 08:18:47 BST 2019


cullmann added inline comments.

INLINE COMMENTS

> dhaumann wrote in katedocument.cpp:2594
> Shall we wrap all this with
> 
>   if (!(backupSuffix.isEmpty() && backupPrefix.isEmpty())) {...}

Why not, saves useless evals.

REPOSITORY
  R39 KTextEditor

BRANCH
  bug

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

To: dhaumann, cullmann
Cc: ngraham, kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190911/c656cc05/attachment.html>


More information about the Kde-frameworks-devel mailing list