D29281: Deprecate defunct functions

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Fri May 22 12:22:20 BST 2020


kossebau added inline comments.

INLINE COMMENTS

> abstractrunner.h:154
>           * is called, the runner should return true
> +         * @deprecated Since 5,0, this feature has been defunct
>           */

Should be "5.0", not "5,0" here in the API dox text and elsewhere, no? (dot instead of comma)

> abstractrunner.h:156
>           */
> +        KRUNNER_DEPRECATED_VERSION(5, 71, "No longer use, feature removed")
>          bool hasRunOptions();

ECMGenerateExportHeader now (for 5.71) features support for using a different version number to be shown in the compiler warning: KRUNNER_DEPRECATED_VERSION_BELATED

So you could make this:

  KRUNNER_DEPRECATED_VERSION_BELATED(5, 71,  5, 0, "No longer use, feature removed")

REPOSITORY
  R308 KRunner

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

To: alex, #plasma, broulik, davidedmundson, vkrause, meven
Cc: kossebau, 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/20200522/5c630f6f/attachment.htm>


More information about the Kde-frameworks-devel mailing list