D23523: [SlaveBase] Use QMap instead of KConfig to store ioslave config

Méven Car noreply at phabricator.kde.org
Wed May 13 14:29:34 BST 2020


meven added inline comments.

INLINE COMMENTS

> kossebau wrote in slavebase.h:355
> Given there are still a few usages of config() left which seem not easily replaceable, it would be better to remove the deprecation tag for the compiler, to not have false warnings on those places (see e.g. http.cpp for certain usages still).
> A deprecation tag should be only set if there is a full migration path available ideally, otherwise people will run into warnings they cannot do something about.

Agreed, perhaps update the documentation slightly

REPOSITORY
  R241 KIO

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

To: meven, davidedmundson, dfaure, #frameworks
Cc: kossebau, broulik, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200513/e2921973/attachment.htm>


More information about the Kde-frameworks-devel mailing list