D23832: Variable expansion: Prefer return value over return argument

[Dominik Haumann]

dhaumann added a comment.


  KateDialogs contains the following:
  
    if (uiadv->edtBackupSuffix->text().isEmpty() && 
uiadv->edtBackupPrefix->text().isEmpty()) {
        KMessageBox::information(
            this,
            i18n("You did not provide a backup suffix or prefix. Using default 
suffix: '~'"),
            i18n("No Backup Suffix or Prefix")
        );
        uiadv->edtBackupSuffix->setText(QStringLiteral("~"));
    }
  
  Theoretically, if a variable exands to nothing (JS:..., ENV:...) we could 
still run into both empty prefix and suffix in katedocument.cpp.

REPOSITORY
  R39 KTextEditor

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

To: dhaumann
Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, 
ngraham, bruns, demsking, cullmann, sars, dhaumann