D5627: Remove KDELibs4Support from KSudoku

[Johan Ouwerkerk]

ouwerkerk added a comment.


  In https://phabricator.kde.org/D5627#105875, @ltoscano wrote:
  
  > I wonder if instead of all the if (), can't we just change the signatures to pass a real QString reference instead of a pointer?
  
  
  Yes we can. In that case the callers need to be similar to:
  
    const QString errorMsg;
    const auto ptr = doTheCall(..., errorMsg);
    if(!ptr) {
      KMessageBox ....
    }

REPOSITORY
  R417 KSudoku

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

To: stikonas, #kde_games, ltoscano, ouwerkerk
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20170429/9513e60b/attachment.html>