dhaumann added inline comments.

INLINE COMMENTS

> katedocument.cpp:2594
> +    const auto backupSuffix = 
> KTextEditor::EditorPrivate::self()->variableExpansionManager()->expandText(config()->backupSuffix(),
>  nullptr);
> +    if (backupPrefix.contains(QDir::separator())) {
>          /**

Shall we wrap all this with

  if (!(backupSuffix.isEmpty() && backupPrefix.isEmpty())) {...}

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

Reply via email to