D23979: Add missing parameters to Permission related jobs and fix url typo.

Daniel Vrátil noreply at phabricator.kde.org
Tue Sep 17 13:04:52 BST 2019


dvratil requested changes to this revision.
dvratil added a comment.
This revision now requires changes to proceed.


  Thanks for the fix. Could you also modify the constructors so that the class members are initialized using the `fooDefault` constants as well, so that we really only have a single place in the entire file where the default value is defined?

REPOSITORY
  R477 KGAPI Library

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

To: barchiesi, dvratil
Cc: #libkgapi, kde-pim, barchiesi, fbampaloukas, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190917/e3548413/attachment.html>


More information about the kde-pim mailing list