[Differential] [Updated] D4692: Fix deprecation warning for KRun::runUrl()

Ragnar Thomsen noreply at phabricator.kde.org
Tue Feb 21 19:17:38 UTC 2017


rthomsen marked an inline comment as done.
rthomsen added inline comments.

INLINE COMMENTS

> elvisangelaccio wrote in batchextract.cpp:173
> You can remove the last two arguments, they already take a default value

If they are removed the old overload will be used and the deprecation warning emitted.

> elvisangelaccio wrote in part.cpp:1265
> Same here, QString() and QByteArray not necessary

Same as above.

REPOSITORY
  R36 Ark

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: rthomsen, elvisangelaccio
Cc: kde-utils-devel, #ark, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20170221/6e64b846/attachment.html>


More information about the Kde-utils-devel mailing list