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

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Wed May 13 12:47:39 BST 2020


kossebau added inline comments.

INLINE COMMENTS

> slavebase.h:355
>       */
> -    KConfigGroup *config();
> +    KIOCORE_DEPRECATED KConfigGroup *config();
>  

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.

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/3f25caad/attachment.htm>


More information about the Kde-frameworks-devel mailing list